Skip to content

Commit

Permalink
Problem: different maxGasWanted config leads hash mismatch (#560)
Browse files Browse the repository at this point in the history
this bug was introduced in 773438f
  • Loading branch information
mmsqe authored Nov 12, 2024
1 parent c9e06fa commit 64bd2de
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,7 @@ Ref: https://keepachangelog.com/en/1.0.0/
* (rpc) [#545](https://github.com/crypto-org-chain/ethermint/pull/545) Fix state overwrite in debug trace APIs.
* (rpc) [#554](https://github.com/crypto-org-chain/ethermint/pull/554) No trace detail on insufficient balance.
* (rpc) [#558](https://github.com/crypto-org-chain/ethermint/pull/558) New tracer in predecessors to trace balance correctly when `debug_traceTransaction`.
* (ante) [#560](https://github.com/crypto-org-chain/ethermint/pull/560) Check gasWanted only in checkTx mode.

### Improvements

Expand Down
2 changes: 1 addition & 1 deletion app/ante/eth.go
Original file line number Diff line number Diff line change
Expand Up @@ -146,7 +146,7 @@ func CheckEthGasConsume(

// We can't trust the tx gas limit, because we'll refund the unused gas.
gasLimit := msgEthTx.GetGas()
if maxGasWanted != 0 {
if ctx.IsCheckTx() && maxGasWanted != 0 {
gasLimit = min(gasLimit, maxGasWanted)
}
if gasWanted > math.MaxInt64-gasLimit {
Expand Down

0 comments on commit 64bd2de

Please sign in to comment.