Restrict data gas prechecks to only txs with data

Inphi · Inphi · commit dc3740cb7f4b · 2022-10-07T17:54:27.000-07:00
diff --git a/core/state_transition.go b/core/state_transition.go
@@ -294,7 +294,8 @@ func (st *StateTransition) preCheck() error {
 			}
 		}
 	}
-	if st.evm.ChainConfig().IsSharding(st.evm.Context.BlockNumber) {
+	usesDataGas := len(st.msg.DataHashes()) != 0
+	if usesDataGas && st.evm.ChainConfig().IsSharding(st.evm.Context.BlockNumber) {
 		dataGasPrice := misc.GetDataGasPrice(st.evm.Context.ExcessDataGas)
 		if dataGasPrice.Cmp(st.maxFeePerDataGas) > 0 {
 			return fmt.Errorf("%w: address %v, maxFeePerDataGas: %v dataGasPrice: %v, excessDataGas: %v",

Original file line number	Diff line number	Diff line change
`@@ -294,7 +294,8 @@ func (st *StateTransition) preCheck() error {`
`294`	`294`	`}`
`295`	`295`	`}`
`296`	`296`	`}`
`297`		`- if st.evm.ChainConfig().IsSharding(st.evm.Context.BlockNumber) {`
	`297`	`+ usesDataGas := len(st.msg.DataHashes()) != 0`
	`298`	`+ if usesDataGas && st.evm.ChainConfig().IsSharding(st.evm.Context.BlockNumber) {`
`298`	`299`	`dataGasPrice := misc.GetDataGasPrice(st.evm.Context.ExcessDataGas)`
`299`	`300`	`if dataGasPrice.Cmp(st.maxFeePerDataGas) > 0 {`
`300`	`301`	`return fmt.Errorf("%w: address %v, maxFeePerDataGas: %v dataGasPrice: %v, excessDataGas: %v",`