fix: gas config is set to zero in ibc transfer keeper

CHANGELOG.md

@@ -11,6 +11,7 @@
 - [\#811](https://github.com/cosmos/evm/pull/811) Use sdk's DefaultBondDenom for default evm denom in genesis.
 - [\#823](https://github.com/cosmos/evm/pull/823) Remove authz dependency from test suite and EvmApp interface
 - [\#829](https://github.com/cosmos/evm/pull/829) Seperate test app interface
+- [\#833]() Use normal gas config in ibc transfer to prevent potential DoS attack
 
 ### FEATURES
 

x/ibc/transfer/keeper/msg_server.go

@@ -10,8 +10,6 @@ import (
 	erc20types "github.com/cosmos/evm/x/erc20/types"
 	"github.com/cosmos/ibc-go/v10/modules/apps/transfer/types"
 
-	storetypes "cosmossdk.io/store/types"
-
 	"github.com/cosmos/cosmos-sdk/telemetry"
 	sdk "github.com/cosmos/cosmos-sdk/types"
 )
@@ -27,21 +25,6 @@ var _ types.MsgServer = Keeper{}
 func (k Keeper) Transfer(goCtx context.Context, msg *types.MsgTransfer) (*types.MsgTransferResponse, error) {
 	ctx := sdk.UnwrapSDKContext(goCtx)
 
-	// Temporarily save the KV and transient KV gas config. To avoid extra costs for relayers
-	// these two gas config are replaced with empty one and should be restored before exiting this function.
-	kvGasCfg := ctx.KVGasConfig()
-	transientKVGasCfg := ctx.TransientKVGasConfig()
-	ctx = ctx.
-		WithKVGasConfig(storetypes.GasConfig{}).
-		WithTransientKVGasConfig(storetypes.GasConfig{})
-
-	defer func() {
-		// Return the KV gas config to initial values
-		ctx = ctx.
-			WithKVGasConfig(kvGasCfg).
-			WithTransientKVGasConfig(transientKVGasCfg)
-	}()
-
 	// use native denom or contract address
 	denom := strings.TrimPrefix(msg.Token.Denom, erc20types.Erc20NativeCoinDenomPrefix)