diff options
author | Péter Szilágyi <peterke@gmail.com> | 2017-08-25 17:30:51 +0800 |
---|---|---|
committer | Péter Szilágyi <peterke@gmail.com> | 2017-08-25 18:00:27 +0800 |
commit | 08f27428b4e97c339a220b7155ad13f4ef5a6767 (patch) | |
tree | 8decfb111ffe4d1c409f56e2d0dd660f01e77e19 /core/vm | |
parent | 27a5622e995d762683dd1a79423d83fcf3e62ccf (diff) | |
download | dexon-08f27428b4e97c339a220b7155ad13f4ef5a6767.tar dexon-08f27428b4e97c339a220b7155ad13f4ef5a6767.tar.gz dexon-08f27428b4e97c339a220b7155ad13f4ef5a6767.tar.bz2 dexon-08f27428b4e97c339a220b7155ad13f4ef5a6767.tar.lz dexon-08f27428b4e97c339a220b7155ad13f4ef5a6767.tar.xz dexon-08f27428b4e97c339a220b7155ad13f4ef5a6767.tar.zst dexon-08f27428b4e97c339a220b7155ad13f4ef5a6767.zip |
core, tests: implement Metropolis EIP 684
Diffstat (limited to 'core/vm')
-rw-r--r-- | core/vm/errors.go | 11 | ||||
-rw-r--r-- | core/vm/evm.go | 14 |
2 files changed, 17 insertions, 8 deletions
diff --git a/core/vm/errors.go b/core/vm/errors.go index 69c7d6a98..b19366be0 100644 --- a/core/vm/errors.go +++ b/core/vm/errors.go @@ -19,9 +19,10 @@ package vm import "errors" var ( - ErrOutOfGas = errors.New("out of gas") - ErrCodeStoreOutOfGas = errors.New("contract creation code storage out of gas") - ErrDepth = errors.New("max call depth exceeded") - ErrTraceLimitReached = errors.New("the number of logs reached the specified limit") - ErrInsufficientBalance = errors.New("insufficient balance for transfer") + ErrOutOfGas = errors.New("out of gas") + ErrCodeStoreOutOfGas = errors.New("contract creation code storage out of gas") + ErrDepth = errors.New("max call depth exceeded") + ErrTraceLimitReached = errors.New("the number of logs reached the specified limit") + ErrInsufficientBalance = errors.New("insufficient balance for transfer") + ErrContractAddressCollision = errors.New("contract address collision") ) diff --git a/core/vm/evm.go b/core/vm/evm.go index 34933a1dc..caf8b4507 100644 --- a/core/vm/evm.go +++ b/core/vm/evm.go @@ -25,6 +25,10 @@ import ( "github.com/ethereum/go-ethereum/params" ) +// emptyCodeHash is used by create to ensure deployment is disallowed to already +// deployed contract addresses (relevant after the account abstraction). +var emptyCodeHash = crypto.Keccak256Hash(nil) + type ( CanTransferFunc func(StateDB, common.Address, *big.Int) bool TransferFunc func(StateDB, common.Address, common.Address, *big.Int) @@ -307,13 +311,17 @@ func (evm *EVM) Create(caller ContractRef, code []byte, gas uint64, value *big.I if !evm.CanTransfer(evm.StateDB, caller.Address(), value) { return nil, common.Address{}, gas, ErrInsufficientBalance } - - // Create a new account on the state + // Ensure there's no existing contract already at the designated address nonce := evm.StateDB.GetNonce(caller.Address()) evm.StateDB.SetNonce(caller.Address(), nonce+1) - snapshot := evm.StateDB.Snapshot() contractAddr = crypto.CreateAddress(caller.Address(), nonce) + contractHash := evm.StateDB.GetCodeHash(contractAddr) + if evm.StateDB.GetNonce(contractAddr) != 0 || (contractHash != (common.Hash{}) && contractHash != emptyCodeHash) { + return nil, common.Address{}, 0, ErrContractAddressCollision + } + // Create a new account on the state + snapshot := evm.StateDB.Snapshot() evm.StateDB.CreateAccount(contractAddr) if evm.ChainConfig().IsEIP158(evm.BlockNumber) { evm.StateDB.SetNonce(contractAddr, 1) |