Skip to content

Commit 6ccd2c6

Browse files
committed
Remove temp fix for gas estimation failures with SSTORE refunds
1 parent b30930d commit 6ccd2c6

File tree

1 file changed

+1
-7
lines changed

1 file changed

+1
-7
lines changed

services/requester/requester.go

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@ import (
2121
gethCore "github.com/onflow/go-ethereum/core"
2222
"github.com/onflow/go-ethereum/core/types"
2323
gethVM "github.com/onflow/go-ethereum/core/vm"
24-
"github.com/onflow/go-ethereum/params"
2524
"github.com/rs/zerolog"
2625
"golang.org/x/sync/errgroup"
2726

@@ -381,12 +380,7 @@ func (e *EVM) EstimateGas(
381380
return 0, getErrorForCode(evmResult.ErrorCode)
382381
}
383382

384-
// This minimum gas availability is needed for:
385-
// https://github.com/onflow/go-ethereum/blob/master/core/vm/operations_acl.go#L29-L32
386-
// Note that this is not actually consumed in the end.
387-
// TODO: Consider moving this to `EVM.dryRun`, if we want the
388-
// fix to also apply for the EVM API, on Cadence side.
389-
gasConsumed := evmResult.GasConsumed + params.SstoreSentryGasEIP2200 + 1
383+
gasConsumed := evmResult.GasConsumed
390384

391385
e.logger.Debug().
392386
Uint64("gas", gasConsumed).

0 commit comments

Comments
 (0)