status-im
/
nimbus-eth1
mirror of https://github.com/status-im/nimbus-eth1.git
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

Don't touch nonce until after address in-use check

This commit is contained in:
coffeepots 2018-09-04 17:15:46 +01:00
parent 555687ba97
commit ca6eb0cb38
1 changed files with 11 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
nimbus/vm/interpreter/opcodes_impl.nim
View File

@ -546,22 +546,21 @@ op create, inline = false, value, startPosition, size:
## TODO dynamic gas that depends on remaining gas
var
contractAddress: EthAddress
isCollision: bool
computation.vmState.mutateStateDB:
let creationNonce = db.getNonce(computation.msg.storageAddress)
db.setNonce(computation.msg.storageAddress, creationNonce + 1)
let
readOnlyState = computation.vmState.readOnlyStateDB
creationNonce = readOnlyState.getNonce(computation.msg.storageAddress)
contractAddress = generateAddress(computation.msg.storageAddress, creationNonce)
# Regarding collisions, see: https://github.com/status-im/nimbus/issues/133
isCollision = db.hasCodeOrNonce(contractAddress)
isCollision = readOnlyState.hasCodeOrNonce(contractAddress)
if isCollision:
# Regarding collisions, see: https://github.com/status-im/nimbus/issues/133
# See: https://github.com/ethereum/EIPs/issues/684
if not isCollision:
computation.vmState.mutateStateDB:
db.setNonce(computation.msg.storageAddress, creationNonce + 1)
else:
debug("Address collision while creating contract", address = contractAddress.toHex)
push: 0
return
raise newException(ValidationError, "Contract address already exists and is in use")
let childMsg = prepareChildMessage(
computation,