Merge pull request #996 from karalabe/fix-potential-crosscheck-race

eth/downloader: circumvent download race between crosscheck and hashes

1 files changed, 7 insertions, 5 deletions

diff --git a/eth/downloader/downloader.go b/eth/downloader/downloader.go
index a0a5b20a2..1bc81406c 100644
--- a/eth/downloader/downloader.go
+++ b/eth/downloader/downloader.go
@@ -284,12 +284,14 @@ func (d *Downloader) fetchHashes(p *peer, h common.Hash) error {
 			}
 			if !done {
 				// Try and fetch a random block to verify the hash batch
-				cross := inserts[rand.Intn(len(inserts))]
-				glog.V(logger.Detail).Infof("Cross checking (%s) with %x", active.id, cross)
-
-				d.checks[cross] = time.Now().Add(blockTTL)
-				active.getBlocks([]common.Hash{cross})
+				// Skip the last hash as the cross check races with the next hash fetch
+				if len(inserts) > 1 {
+					cross := inserts[rand.Intn(len(inserts)-1)]
+					glog.V(logger.Detail).Infof("Cross checking (%s) with %x", active.id, cross)
 
+					d.checks[cross] = time.Now().Add(blockTTL)
+					active.getBlocks([]common.Hash{cross})
+				}
 				// Also fetch a fresh
 				active.getHashes(head)
 				continue