From 09d889d2e3405f3c15d0b92fc0f027bed232a6e1 Mon Sep 17 00:00:00 2001 From: Shude Li Date: Fri, 16 Aug 2024 22:33:41 +0800 Subject: [PATCH] core: remove withdrawal length check for state processor (#30286) The withdrawal length is already verified by the beacon consensus package, so the check in the state processor is a duplicate. --- core/state_processor.go | 6 ------ 1 file changed, 6 deletions(-) diff --git a/core/state_processor.go b/core/state_processor.go index c21f644f98..cc41583165 100644 --- a/core/state_processor.go +++ b/core/state_processor.go @@ -17,7 +17,6 @@ package core import ( - "errors" "fmt" "math/big" @@ -93,11 +92,6 @@ func (p *StateProcessor) Process(block *types.Block, statedb *state.StateDB, cfg receipts = append(receipts, receipt) allLogs = append(allLogs, receipt.Logs...) } - // Fail if Shanghai not enabled and len(withdrawals) is non-zero. - withdrawals := block.Withdrawals() - if len(withdrawals) > 0 && !p.config.IsShanghai(block.Number(), block.Time()) { - return nil, nil, 0, errors.New("withdrawals before shanghai") - } // Finalize the block, applying any consensus engine specific extras (e.g. block rewards) p.chain.engine.Finalize(p.chain, header, statedb, block.Body())