Skip to content

Commit 2cfbabe

Browse files
committed
Bandit exclusions
- bandit changed how B311 works so we need to annotate accordingly
1 parent 79268cb commit 2cfbabe

File tree

4 files changed

+9
-8
lines changed

4 files changed

+9
-8
lines changed

src/helper_random.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ def randomsample(population, k):
3535
without replacement, its called
3636
partial shuffle.
3737
"""
38-
return random.sample(population, k)
38+
return random.sample(population, k) # nosec B311
3939

4040

4141
def randomrandrange(x, y=None):

src/network/asyncore_pollchoose.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -233,13 +233,13 @@ def select_poller(timeout=0.0, map=None):
233233
if err.args[0] in (WSAENOTSOCK, ):
234234
return
235235

236-
for fd in random.sample(r, len(r)):
236+
for fd in random.sample(r, len(r)): # nosec B311
237237
obj = map.get(fd)
238238
if obj is None:
239239
continue
240240
read(obj)
241241

242-
for fd in random.sample(w, len(w)):
242+
for fd in random.sample(w, len(w)): # nosec B311
243243
obj = map.get(fd)
244244
if obj is None:
245245
continue
@@ -297,7 +297,7 @@ def poll_poller(timeout=0.0, map=None):
297297
except socket.error as err:
298298
if err.args[0] in (EBADF, WSAENOTSOCK, EINTR):
299299
return
300-
for fd, flags in random.sample(r, len(r)):
300+
for fd, flags in random.sample(r, len(r)): # nosec B311
301301
obj = map.get(fd)
302302
if obj is None:
303303
continue
@@ -357,7 +357,7 @@ def epoll_poller(timeout=0.0, map=None):
357357
if err.args[0] != EINTR:
358358
raise
359359
r = []
360-
for fd, flags in random.sample(r, len(r)):
360+
for fd, flags in random.sample(r, len(r)): # nosec B311
361361
obj = map.get(fd)
362362
if obj is None:
363363
continue
@@ -420,7 +420,7 @@ def kqueue_poller(timeout=0.0, map=None):
420420

421421
events = kqueue_poller.pollster.control(updates, selectables, timeout)
422422
if len(events) > 1:
423-
events = random.sample(events, len(events))
423+
events = random.sample(events, len(events)) # nosec B311
424424

425425
for event in events:
426426
fd = event.ident

src/network/dandelion.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -198,7 +198,7 @@ def reRandomiseStems(self):
198198
with self.lock:
199199
try:
200200
# random two connections
201-
self.stem = sample(
201+
self.stem = sample( # nosec B311
202202
self.pool.outboundConnections.values(), MAX_STEMS)
203203
# not enough stems available
204204
except ValueError:

src/network/tcp.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -200,7 +200,8 @@ def sendAddr(self):
200200
elemCount = min(
201201
len(filtered),
202202
maxAddrCount / 2 if n else maxAddrCount)
203-
addrs[s] = random.sample(filtered, elemCount)
203+
addrs[s] = random.sample(filtered,
204+
elemCount) # nosec B311
204205
for substream in addrs:
205206
for peer, params in addrs[substream]:
206207
templist.append((substream, peer, params["lastseen"]))

0 commit comments

Comments
 (0)