Skip to content

Commit

Permalink
fix
Browse files Browse the repository at this point in the history
  • Loading branch information
yihuang committed Sep 23, 2024
1 parent c8d3113 commit bdbb4a2
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions x/evm/keeper/state_transition.go
Original file line number Diff line number Diff line change
Expand Up @@ -324,28 +324,29 @@ func (k *Keeper) ApplyMessageWithConfig(

// Allow the tracer captures the tx level events, mainly the gas consumption.
leftoverGas := msg.GasLimit
sender := sdk.AccAddress(msg.From.Bytes())
senderAddr := sdk.AccAddress(msg.From.Bytes())
if cfg.Tracer != nil {
if cfg.DebugTrace {
// msg.GasPrice should have been set to effective gas price
amount := new(big.Int).Mul(msg.GasPrice, new(big.Int).SetUint64(msg.GasLimit))
if err := k.SubBalance(ctx, sender, sdk.NewCoins(sdk.NewCoin(cfg.Params.EvmDenom, sdkmath.NewIntFromBigInt(amount)))); err != nil {
if err := k.SubBalance(ctx, senderAddr, sdk.NewCoins(sdk.NewCoin(cfg.Params.EvmDenom, sdkmath.NewIntFromBigInt(amount)))); err != nil {
return nil, errorsmod.Wrap(err, "failed to subtract balance")
}
if err := k.incrNonce(ctx, sender); err != nil {
if err := k.incrNonce(ctx, senderAddr); err != nil {
return nil, errorsmod.Wrap(err, "failed to increment nonce")
}
}
cfg.Tracer.CaptureTxStart(leftoverGas)
defer func() {
if cfg.DebugTrace {
amount := new(big.Int).Mul(msg.GasPrice, new(big.Int).SetUint64(leftoverGas))
k.AddBalance(ctx, sender, sdk.NewCoins(sdk.NewCoin(cfg.Params.EvmDenom, sdkmath.NewIntFromBigInt(amount))))
k.AddBalance(ctx, senderAddr, sdk.NewCoins(sdk.NewCoin(cfg.Params.EvmDenom, sdkmath.NewIntFromBigInt(amount))))

Check failure on line 343 in x/evm/keeper/state_transition.go

View workflow job for this annotation

GitHub Actions / Run golangci-lint

Error return value of `k.AddBalance` is not checked (errcheck)
}
cfg.Tracer.CaptureTxEnd(leftoverGas)
}()
}

sender := vm.AccountRef(msg.From)
stateDB := statedb.NewWithParams(ctx, k, cfg.TxConfig, cfg.Params.EvmDenom)
var evm *vm.EVM
if cfg.Overrides != nil {
Expand Down

0 comments on commit bdbb4a2

Please sign in to comment.