fix lint

x/cronos/keeper/precompiles/bank.go

@@ -19,6 +19,10 @@ import (
 
 const (
 	EVMDenomPrefix = "evm/"
+	MintName       = "mint"
+	BurnName       = "burn"
+	BalanceOfName  = "balanceOf"
+	TransferName   = "transfer"
 )
 
 var (
@@ -58,11 +62,11 @@ func (bc *BankContract) RequiredGas(input []byte) uint64 {
 		return 0
 	}
 	switch method.Name {
-	case "mint", "burn":
+	case MintName, BurnName:
 		return 1000
-	case "balanceOf":
+	case BalanceOfName:
 		return 1000
-	case "transfer":
+	case TransferName:
 		return 1000
 	default:
 		return 0
@@ -94,7 +98,7 @@ func (bc *BankContract) Run(evm *vm.EVM, contract *vm.Contract, readonly bool) (
 	stateDB := evm.StateDB.(ExtStateDB)
 	precompileAddr := bc.Address()
 	switch method.Name {
-	case "mint", "burn":
+	case MintName, BurnName:
 		if readonly {
 			return nil, errors.New("the method is not readonly")
 		}
@@ -138,7 +142,7 @@ func (bc *BankContract) Run(evm *vm.EVM, contract *vm.Contract, readonly bool) (
 			return nil, err
 		}
 		return method.Outputs.Pack(true)
-	case "balanceOf":
+	case BalanceOfName:
 		args, err := method.Inputs.Unpack(contract.Input[4:])
 		if err != nil {
 			return nil, errors.New("fail to unpack input arguments")
@@ -148,7 +152,7 @@ func (bc *BankContract) Run(evm *vm.EVM, contract *vm.Contract, readonly bool) (
 		// query from storage
 		balance := bc.bankKeeper.GetBalance(stateDB.CacheContext(), sdk.AccAddress(addr.Bytes()), EVMDenom(token)).Amount.BigInt()
 		return method.Outputs.Pack(balance)
-	case "transfer":
+	case TransferName:
 		if readonly {
 			return nil, errors.New("the method is not readonly")
 		}

x/cronos/keeper/precompiles/ica.go

@@ -19,8 +19,11 @@ import (
 	"github.com/ethereum/go-ethereum/core/vm"
 )
 
-// TODO: Replace this const with adjusted gas cost corresponding to input when executing precompile contract.
-const ICAContractRequiredGas = 10000
+const (
+	RegisterAccountName = "registerAccount"
+	QueryAccountName    = "queryAccount"
+	SubmitMsgsName      = "submitMsgs"
+)
 
 var (
 	icaABI             abi.ABI
@@ -60,11 +63,11 @@ func (ic *IcaContract) RequiredGas(input []byte) uint64 {
 		return 0
 	}
 	switch method.Name {
-	case "registerAccount":
+	case RegisterAccountName:
 		return 1000
-	case "queryAccount":
+	case QueryAccountName:
 		return 1000
-	case "submitMsgs":
+	case SubmitMsgsName:
 		return 1000
 	default:
 		return 0
@@ -90,7 +93,7 @@ func (ic *IcaContract) Run(evm *vm.EVM, contract *vm.Contract, readonly bool) ([
 	converter := cronosevents.IcaConvertEvent
 	var execErr error
 	switch method.Name {
-	case "registerAccount":
+	case RegisterAccountName:
 		if readonly {
 			return nil, errors.New("the method is not readonly")
 		}
@@ -112,7 +115,7 @@ func (ic *IcaContract) Run(evm *vm.EVM, contract *vm.Contract, readonly bool) ([
 			return nil, execErr
 		}
 		return method.Outputs.Pack(true)
-	case "queryAccount":
+	case QueryAccountName:
 		args, err := method.Inputs.Unpack(contract.Input[4:])
 		if err != nil {
 			return nil, errors.New("fail to unpack input arguments")
@@ -134,7 +137,7 @@ func (ic *IcaContract) Run(evm *vm.EVM, contract *vm.Contract, readonly bool) ([
 			icaAddress = response.InterchainAccountAddress
 		}
 		return method.Outputs.Pack(icaAddress)
-	case "submitMsgs":
+	case SubmitMsgsName:
 		if readonly {
 			return nil, errors.New("the method is not readonly")
 		}

x/cronos/keeper/precompiles/relayer.go

@@ -35,6 +35,9 @@ func (bc *RelayerContract) Address() common.Address {
 
 // RequiredGas calculates the contract gas use
 func (bc *RelayerContract) RequiredGas(input []byte) uint64 {
+	if len(input) < prefixSize4Bytes {
+		return 0
+	}
 	prefix := int(binary.LittleEndian.Uint32(input[:prefixSize4Bytes]))
 	switch prefix {
 	case prefixCreateClient: