status-im/nimbus-eth2
2
0
Fork 0
mirror of https://github.com/status-im/nimbus-eth2.git synced 2025-02-23 11:48:33 +00:00
Code Issues Projects Releases Wiki Activity

Remove std/random again

Browse Source
This commit is contained in:
Zahary Karadjov 2021-01-21 19:39:04 +02:00
parent 718feef802
commit 960666d1ed
No known key found for this signature in database
GPG Key ID: C8936F8A3073D609
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
beacon_chain/nimbus_beacon_node.nim
View File

@ -8,7 +8,6 @@
import import
# Standard library # Standard library
std/[math, os, strformat, strutils, tables, times, std/[math, os, strformat, strutils, tables, times,
random, # TODO: review usage of Nim RNG
terminal, osproc], terminal, osproc],
system/ansi_c, system/ansi_c,
@ -656,12 +655,11 @@ proc setupSelfSlashingProtection(node: BeaconNode, slot: Slot) =
# 2 epochs is dangerous because it'll guarantee colliding probes in the # 2 epochs is dangerous because it'll guarantee colliding probes in the
# overlapping case. # overlapping case.
# TODO: is a pseudo-random RNG enough here? let rng = node.network.rng
# If so, should we store the state explicitly?
# So dPE == 2 -> epoch + 1, always; dPE == 3 -> epoch + (1 or 2), etc. # So dPE == 2 -> epoch + 1, always; dPE == 3 -> epoch + (1 or 2), etc.
node.processor.gossipSlashingProtection.probeEpoch = node.processor.gossipSlashingProtection.probeEpoch =
slot.epoch + 1 + rand(duplicateValidatorEpochs.int - 2).uint64 slot.epoch + 1 + rng[].rand(duplicateValidatorEpochs.int - 2).uint64
doAssert node.processor.gossipSlashingProtection.probeEpoch < doAssert node.processor.gossipSlashingProtection.probeEpoch <
node.processor.gossipSlashingProtection.broadcastStartEpoch node.processor.gossipSlashingProtection.broadcastStartEpoch