From 06c383f8d97b3981ee38b9148ceb745aecc94542 Mon Sep 17 00:00:00 2001 From: Vinod Damle Date: Fri, 22 Jan 2021 15:25:11 -0500 Subject: [PATCH] core: fix missing receipt ref: https://github.com/ethereum/go-ethereum/pull/21045 --- core/blockchain.go | 14 ++++++++++++++ 1 file changed, 14 insertions(+) diff --git a/core/blockchain.go b/core/blockchain.go index 71cb48d5db..34a91b8a3f 100644 --- a/core/blockchain.go +++ b/core/blockchain.go @@ -1738,6 +1738,20 @@ func (bc *BlockChain) insertChain(chain types.Blocks, verifySeals bool) (int, [] "uncles", len(block.Uncles()), "txs", len(block.Transactions()), "gas", block.GasUsed(), "root", block.Root()) + // Special case. Commit the empty receipt slice if we meet the known + // block in the middle. It can only happen in the clique chain. Whenever + // we insert blocks via `insertSideChain`, we only commit `td`, `header` + // and `body` if it's non-existent. Since we don't have receipts without + // reexecution, so nothing to commit. But if the sidechain will be adpoted + // as the canonical chain eventually, it needs to be reexecuted for missing + // state, but if it's this special case here(skip reexecution) we will lose + // the empty receipt entry. + if len(block.Transactions()) == 0 { + rawdb.WriteReceipts(bc.db, block.Hash(), block.NumberU64(), nil) + } else { + log.Error("Please file an issue, skip known block execution without receipt", + "hash", block.Hash(), "number", block.NumberU64()) + } if err := bc.writeKnownBlock(block); err != nil { return it.index, nil, nil, err }