reject tx if address in blacklist

CHANGELOG.md

@@ -50,6 +50,7 @@ Ref: https://keepachangelog.com/en/1.0.0/
 * (api) [\#687](https://github.com/cosmos/ethermint/issues/687) Returns error for a transaction with an incorrect nonce. 
 * (evm) [\#674](https://github.com/cosmos/ethermint/issues/674) Reset all cache after account data has been committed in `EndBlock` to make sure every node state consistent.
 * (evm) [\#672](https://github.com/cosmos/ethermint/issues/672) Fix panic of `wrong Block.Header.AppHash` when restart a node with snapshot.
+* (evm) [\#753](https://github.com/cosmos/ethermint/issues/753) Using evm tx to transfer token to the module account causes the module account balance to be tainted.
 
 ## [v0.4.0] - 2020-12-15
 

app/ethermint.go

@@ -209,7 +209,7 @@ func NewEthermintApp(
 	)
 	app.UpgradeKeeper = upgrade.NewKeeper(skipUpgradeHeights, keys[upgrade.StoreKey], app.cdc)
 	app.EvmKeeper = evm.NewKeeper(
-		app.cdc, keys[evm.StoreKey], app.subspaces[evm.ModuleName], app.AccountKeeper,
+		app.cdc, keys[evm.StoreKey], app.subspaces[evm.ModuleName], app.AccountKeeper, app.BankKeeper
 	)
 
 	// create evidence keeper with router

x/evm/types/statedb.go

@@ -2,6 +2,7 @@ package types
 
 import (
 	"fmt"
+	"github.com/cosmos/cosmos-sdk/x/bank"
 	"math/big"
 	"sort"
 	"sync"
@@ -45,6 +46,7 @@ type CommitStateDB struct {
 	storeKey      sdk.StoreKey
 	paramSpace    params.Subspace
 	accountKeeper AccountKeeper
+	bankKeeper    bank.Keeper
 
 	// array that hold 'live' objects, which will get modified while processing a
 	// state transition
@@ -90,13 +92,14 @@ type CommitStateDB struct {
 // CONTRACT: Stores used for state must be cache-wrapped as the ordering of the
 // key/value space matters in determining the merkle root.
 func NewCommitStateDB(
-	ctx sdk.Context, storeKey sdk.StoreKey, paramSpace params.Subspace, ak AccountKeeper,
+	ctx sdk.Context, storeKey sdk.StoreKey, paramSpace params.Subspace, ak AccountKeeper, bk bank.Keeper,
 ) *CommitStateDB {
 	return &CommitStateDB{
 		ctx:                  ctx,
 		storeKey:             storeKey,
 		paramSpace:           paramSpace,
 		accountKeeper:        ak,
+		bankKeeper:           bk,
 		stateObjects:         []stateEntry{},
 		addressToObjectIndex: make(map[ethcmn.Address]int),
 		stateObjectsDirty:    make(map[ethcmn.Address]struct{}),
@@ -568,6 +571,11 @@ func (csdb *CommitStateDB) updateStateObject(so *stateObject) error {
 		return fmt.Errorf("invalid balance %s", newBalance)
 	}
 
+	//checking and reject tx if address in blacklist
+	if csdb.bankKeeper.BlacklistedAddr(so.account.GetAddress()) {
+		return fmt.Errorf("invalid address %s", so.account.GetAddress().String())
+	}
+
 	coins := so.account.GetCoins()
 	balance := coins.AmountOf(newBalance.Denom)
 	if balance.IsZero() || !balance.Equal(newBalance.Amount) {