Skip to content

fix: cosmwasm evm query path repeatable undercharged evm exec #18

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
mattkii merged 3 commits into from
Jun 30, 2026
Merged

fix: cosmwasm evm query path repeatable undercharged evm exec #18

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
208efe0
fix: cosmwasm evm query path repeatable undercharged evm exec
mattkii Jun 25, 2026
2a567c7
fix: add changelog entry
mattkii Jun 25, 2026
cdc7c17
fix: bound internal EVM call gas limit by caller gasCap
mattkii Jun 30, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ Follow the [migration document](docs/migrations/v0.5.x_to_v0.6.0.md) for upgrade

### BUG FIXES

- Bound the message `GasLimit` in `CallEVMWithData` to the caller-provided `gasCap` when set (`min(gasCap, DefaultGasCap)`) instead of always using `DefaultGasCap`, so callers (e.g. the CosmWasm ERC20 query bindings) can constrain internal EVM execution to their remaining gas budget.
- Fix EVM fee-abstraction gas refund to compute the refund against the full transaction gas (`gasUsed + leftoverGas`) instead of `gasUsed`, bounding the refund by the paid fee and preventing the fee collector from being drained by high gas limit transactions.
- [\#15](https://github.com/KiiChain/evm/pull/15) Fix distribution precompile 32-byte withdraw address inflating native supply by skipping non-20-byte accounts when mirroring balance changes to the StateDB.

Expand Down
9 changes: 8 additions & 1 deletion x/vm/keeper/call_evm.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,12 +45,19 @@ func (k Keeper) CallEVMWithData(ctx sdk.Context, stateDB *statedb.StateDB, from
return nil, err
}

gasLimit := config.DefaultGasCap
if gasCap != nil && gasCap.IsUint64() {
if capped := gasCap.Uint64(); capped < gasLimit {
gasLimit = capped
}
}

msg := core.Message{
From: from,
To: contract,
Nonce: nonce,
Value: big.NewInt(0),
GasLimit: config.DefaultGasCap,
GasLimit: gasLimit,
GasPrice: big.NewInt(0),
GasTipCap: big.NewInt(0),
GasFeeCap: big.NewInt(0),
Expand Down
Loading