go-pulse/miner
lzhfromustc 62dc59c2bd
miner, test: fix potential goroutine leak (#21989)
In miner/worker.go, there are two goroutine using channel w.newWorkCh: newWorkerLoop() sends to this channel, and mainLoop() receives from this channel. Only the receive operation is in a select.

However, w.exitCh may be closed by another goroutine. This is fine for the receive since receive is in select, but if the send operation is blocking, then it will block forever. This commit puts the send in a select, so it won't block even if w.exitCh is closed.

Similarly, there are two goroutines using channel errc: the parent that runs the test receives from it, and the child created at line 573 sends to it. If the parent goroutine exits too early by calling t.Fatalf() at line 614, then the child goroutine will be blocked at line 574 forever. This commit adds 1 buffer to errc. Now send will not block, and receive is not influenced because receive still needs to wait for the send.
2020-12-11 10:29:42 +01:00
..
miner_test.go miner: fixed race condition in tests (#21664) 2020-10-20 10:58:26 +02:00
miner.go miner: set etherbase even if mining isn't possible at the moment (#21707) 2020-10-14 11:59:11 +02:00
stress_clique.go node: refactor package node (#21105) 2020-08-03 19:40:46 +02:00
stress_ethash.go node: refactor package node (#21105) 2020-08-03 19:40:46 +02:00
unconfirmed_test.go all: remove redundant conversions and import names (#21903) 2020-11-25 21:00:23 +01:00
unconfirmed.go all: replace RWMutex with Mutex in places where RLock is not used (#21622) 2020-10-13 10:58:41 +02:00
worker_test.go eth/downloader: fix spuriously failing tests (#21149) 2020-06-09 11:39:19 +02:00
worker.go miner, test: fix potential goroutine leak (#21989) 2020-12-11 10:29:42 +01:00