diff --git a/core/block_validator.go b/core/block_validator.go index d139b2d8fe..f6d12c1f01 100644 --- a/core/block_validator.go +++ b/core/block_validator.go @@ -59,12 +59,6 @@ func (v *BlockValidator) ValidateBody(block *types.Block) error { if v.bc.HasBlockAndFullState(block.Hash(), block.NumberU64()) { return ErrKnownBlock } - if !v.bc.HasBlockAndFullState(block.ParentHash(), block.NumberU64()-1) { - if !v.bc.HasBlock(block.ParentHash(), block.NumberU64()-1) { - return consensus.ErrUnknownAncestor - } - return consensus.ErrPrunedAncestor - } // Header validity is known at this point, check the uncles and transactions header := block.Header() if err := v.engine.VerifyUncles(v.bc, block); err != nil { @@ -76,6 +70,12 @@ func (v *BlockValidator) ValidateBody(block *types.Block) error { if hash := types.DeriveSha(block.Transactions()); hash != header.TxHash { return fmt.Errorf("transaction root hash mismatch: have %x, want %x", hash, header.TxHash) } + if !v.bc.HasBlockAndFullState(block.ParentHash(), block.NumberU64()-1) { + if !v.bc.HasBlock(block.ParentHash(), block.NumberU64()-1) { + return consensus.ErrUnknownAncestor + } + return consensus.ErrPrunedAncestor + } return nil } diff --git a/eth/downloader/downloader.go b/eth/downloader/downloader.go index e275707fc0..7b62ca4d49 100644 --- a/eth/downloader/downloader.go +++ b/eth/downloader/downloader.go @@ -826,6 +826,7 @@ func (d *Downloader) findAncestor(p *peerConnection, remoteHeader *types.Header) return 0, errBadPeer } start = check + hash = h case <-timeout: p.log.Debug("Waiting for search header timed out", "elapsed", ttl)