This repository was archived by the owner on Nov 30, 2021. It is now read-only.
This repository was archived by the owner on Nov 30, 2021. It is now read-only.
The param 'isEIP2028' of function IntrinsicGas should be true if Istanbul version enabled #730
Description
System info: [Include Ethermint commit, operating system name, and other relevant details]
Steps to reproduce:
the param 'isEIP2028' of function IntrinsicGas which was called in TransitionDb Need reset false to true cause Istanbul version has already be enabled
func (st StateTransition) TransitionDb(ctx sdk.Context, config ChainConfig) (*ExecutionResult, error) {
contractCreation := st.Recipient == nil
//here need reset false to true cause Istanbul version has already be enabled
cost, err := core.IntrinsicGas(st.Payload, contractCreation,true,false)
if err != nil {
return nil, sdkerrors.Wrap(err, "invalid intrinsic gas for transaction")
}
// This gas limit the the transaction gas limit with intrinsic gas subtracted
gasLimit := st.GasLimit - ctx.GasMeter().GasConsumed()Expected behavior: [What you expected to happen]
Actual behavior: [What actually happened]
Additional info: [Include gist of relevant config, logs, etc.]