roysc/plugeth
1
0
Fork 0
forked from cerc-io/plugeth
Code Pull Requests Activity

eth: fix shutdown regression to abort downloads, not just cancel

Browse Source
This commit is contained in:
Péter Szilágyi 2020-04-27 11:22:15 +03:00
parent 1aa83290f5
commit b0bbd47185
No known key found for this signature in database
GPG Key ID: E9AE538CEDF8293D
2 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
eth/downloader/downloader.go
View File

@ -557,6 +557,8 @@ func (d *Downloader) spawnSync(fetchers []func() error) error {
func (d *Downloader) cancel() { func (d *Downloader) cancel() {
// Close the current cancel channel // Close the current cancel channel
d.cancelLock.Lock() d.cancelLock.Lock()
defer d.cancelLock.Unlock()
if d.cancelCh != nil { if d.cancelCh != nil {
select { select {
case <-d.cancelCh: case <-d.cancelCh:
@ -565,7 +567,6 @@ func (d *Downloader) cancel() {
close(d.cancelCh) close(d.cancelCh)
} }
} }
d.cancelLock.Unlock()
} }
// Cancel aborts all of the operations and waits for all download goroutines to // Cancel aborts all of the operations and waits for all download goroutines to

2
eth/sync.go
View File

@ -222,7 +222,7 @@ func (cs *chainSyncer) loop() {
case <-cs.pm.quitSync: case <-cs.pm.quitSync:
if cs.doneCh != nil { if cs.doneCh != nil {
cs.pm.downloader.Cancel() cs.pm.downloader.Terminate() // Double term is fine, Cancel would block until queue is emptied
<-cs.doneCh <-cs.doneCh
} }
return return