fix feesplit innertx bug (#3099)

* fix feesplit innertx bug

* disable ibc and feesplit hook in checktx

x/erc20/keeper/evm_log_handler.go

@@ -100,6 +100,9 @@ func (h SendToIbcEventHandler) EventID() common.Hash {
 // Handle Process the log
 func (h SendToIbcEventHandler) Handle(ctx sdk.Context, contract common.Address, data []byte) error {
 	h.Logger(ctx).Info("trigger evm event", "event", SendToIbcEvent.Name, "contract", contract)
+	if ctx.IsCheckTx() {
+		return nil
+	}
 	// first confirm that the contract address and denom are registered,
 	// to avoid unpacking any contract '__OKCSendToIbc' event, which consumes performance
 	denom, found := h.Keeper.GetDenomByContract(ctx, contract)
@@ -160,6 +163,9 @@ func (h SendNative20ToIbcEventHandler) EventID() common.Hash {
 // Handle Process the log
 func (h SendNative20ToIbcEventHandler) Handle(ctx sdk.Context, contract common.Address, data []byte) error {
 	h.Logger(ctx).Info("trigger evm event", "event", SendNative20ToIbcEvent.Name, "contract", contract)
+	if ctx.IsCheckTx() {
+		return nil
+	}
 	// first confirm that the contract address and denom are registered,
 	// to avoid unpacking any contract '__OKCSendNative20ToIbc' event, which consumes performance
 	denom, found := h.Keeper.GetDenomByContract(ctx, contract)

x/feesplit/keeper/evm_hooks.go

@@ -38,6 +38,9 @@ func (k Keeper) PostTxProcessing(
 	st *evmtypes.StateTransition,
 	receipt *ethtypes.Receipt,
 ) error {
+	if ctx.IsCheckTx() {
+		return nil
+	}
 	// This is different from ibc and wasm, evm tx exists at all times.
 	// in Venus3 height store takes effect,
 	// in Venus3+1 height initGenesis takes effect,
@@ -101,7 +104,9 @@ func (k Keeper) PostTxProcessing(
 	f.HasFee = true
 
 	// add innertx
-	k.addFeesplitInnerTx(receipt.TxHash.Hex(), withdrawer.String(), fees.String())
+	if !ctx.IsCheckTx() {
+		k.addFeesplitInnerTx(receipt.TxHash.Hex(), withdrawer.String(), fees.String())
+	}
 
 	ctx.EventManager().EmitEvents(
 		sdk.Events{