aboutsummaryrefslogtreecommitdiffstats
path: root/eth
diff options
context:
space:
mode:
authorobscuren <geffobscura@gmail.com>2015-05-03 20:11:00 +0800
committerobscuren <geffobscura@gmail.com>2015-05-03 20:11:00 +0800
commitf2a2b2ac70632b878fea393f698168b65adbac2f (patch)
tree18076a2524e8fb8ee2040c1a400030d1dd0c326f /eth
parent493181ea09e5ac1e8798e4e97260fa5bff84ea1c (diff)
downloaddexon-f2a2b2ac70632b878fea393f698168b65adbac2f.tar
dexon-f2a2b2ac70632b878fea393f698168b65adbac2f.tar.gz
dexon-f2a2b2ac70632b878fea393f698168b65adbac2f.tar.bz2
dexon-f2a2b2ac70632b878fea393f698168b65adbac2f.tar.lz
dexon-f2a2b2ac70632b878fea393f698168b65adbac2f.tar.xz
dexon-f2a2b2ac70632b878fea393f698168b65adbac2f.tar.zst
dexon-f2a2b2ac70632b878fea393f698168b65adbac2f.zip
eth/downloader: put back hashes on block overflow error
Diffstat (limited to 'eth')
-rw-r--r--eth/downloader/queue.go33
1 files changed, 14 insertions, 19 deletions
diff --git a/eth/downloader/queue.go b/eth/downloader/queue.go
index 13768229f..1b63a5ffb 100644
--- a/eth/downloader/queue.go
+++ b/eth/downloader/queue.go
@@ -1,6 +1,7 @@
package downloader
import (
+ "fmt"
"math"
"sync"
"time"
@@ -102,17 +103,6 @@ func (c *queue) has(hash common.Hash) bool {
return c.hashPool.Has(hash) || c.fetchPool.Has(hash) || c.blockHashes.Has(hash)
}
-func (c *queue) addBlock(id string, block *types.Block) {
- c.mu.Lock()
- defer c.mu.Unlock()
-
- // when adding a block make sure it doesn't already exist
- if !c.blockHashes.Has(block.Hash()) {
- c.hashPool.Remove(block.Hash())
- c.blocks = append(c.blocks, block)
- }
-}
-
func (c *queue) getBlock(hash common.Hash) *types.Block {
c.mu.Lock()
defer c.mu.Unlock()
@@ -130,7 +120,7 @@ func (c *queue) getBlock(hash common.Hash) *types.Block {
}
// deliver delivers a chunk to the queue that was requested of the peer
-func (c *queue) deliver(id string, blocks []*types.Block) error {
+func (c *queue) deliver(id string, blocks []*types.Block) (err error) {
c.mu.Lock()
defer c.mu.Unlock()
@@ -145,25 +135,30 @@ func (c *queue) deliver(id string, blocks []*types.Block) error {
chunk.peer.ignored.Merge(chunk.hashes)
}
- // seperate the blocks and the hashes
- blockHashes := chunk.fetchedHashes(blocks)
- // merge block hashes
- c.blockHashes.Merge(blockHashes)
// Add the blocks
- for _, block := range blocks {
+ for i, block := range blocks {
// See (1) for future limitation
n := int(block.NumberU64()) - c.blockOffset
if n > len(c.blocks) || n < 0 {
- return errBlockNumberOverflow
+ // set the error and set the blocks which could be processed
+ // abort the rest of the blocks (FIXME this could be improved)
+ err = fmt.Errorf("received block which overflow (N=%v O=%v)", block.Number(), c.blockOffset)
+ blocks = blocks[:i]
+ break
}
c.blocks[n] = block
}
+ // seperate the blocks and the hashes
+ blockHashes := chunk.fetchedHashes(blocks)
+ // merge block hashes
+ c.blockHashes.Merge(blockHashes)
// Add back whatever couldn't be delivered
c.hashPool.Merge(chunk.hashes)
+ // Remove the hashes from the fetch pool
c.fetchPool.Separate(chunk.hashes)
}
- return nil
+ return
}
func (c *queue) alloc(offset, size int) {