flip/dcrd
1
0
Fork 0
mirror of https://github.com/FlipsideCrypto/dcrd.git synced 2026-02-06 10:56:47 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

mining: fix data race

Browse Source 
use a local prng variable
This commit is contained in:
David Hill 2019-09-11 11:11:48 -04:00 committed by Dave Collins
parent 5604ecd689
commit 65a467e0cf
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
mining.go
View File

@ -2026,7 +2026,6 @@ type regenEvent struct {
// stale (e.g. new parent or new votes)
type BgBlkTmplGenerator struct {
wg sync.WaitGroup
prng *rand.Rand
// These fields are provided by the caller when the generator is created and
// are either independently safe for concurrent access or do not change after
@ -2113,7 +2112,6 @@ type BgBlkTmplGenerator struct {
// method to allowing processing.
func newBgBlkTmplGenerator(tg *BlkTmplGenerator, addrs []dcrutil.Address, allowUnsynced bool) *BgBlkTmplGenerator {
return &BgBlkTmplGenerator{
prng: rand.New(rand.NewSource(time.Now().Unix())),
chain: tg.chain,
tg: tg,
allowUnsyncedMining: allowUnsynced,
@ -2470,7 +2468,8 @@ func (g *BgBlkTmplGenerator) genTemplateAsync(ctx context.Context, reason templa
// Pick a mining address at random and generate a block template that
// pays to it.
payToAddr := g.miningAddrs[g.prng.Intn(len(g.miningAddrs))]
prng := rand.New(rand.NewSource(time.Now().Unix()))
payToAddr := g.miningAddrs[prng.Intn(len(g.miningAddrs))]
template, err := g.tg.NewBlockTemplate(payToAddr)
// NOTE: err is handled below.