Re:Security Alert: Ethereum Constantinople Postponement
by api on Tuesday January 15, 2019 @08:45PM
(#18917594)
The whole Ethereum VM is too complex. For a role like smart contracts you'd want something dirt simple with few operations and provability.
[ Parent
| on HN ]
Re:Security Alert: Ethereum Constantinople Postponement
by aakilfernandes on Tuesday January 15, 2019 @08:51PM
(#18917622)
IMHO the VM here isn't really the problem. Its the compiler. The way I look at it this is really a compiler bug, that only gets "activated" with this upgrade.
Re:Security Alert: Ethereum Constantinople Postponement
by amluto on Tuesday January 15, 2019 @09:15PM
(#18917734)
The whole design is bad. Calls out to untrusted code should never have been allowed. You should be able to send messages out to untrusted code and to send ether. Perhaps the recipient of a message could be allowed to raise an exception, thus causing the transaction to abort, but otherwise the untrusted code should never have been allowed to reenter anything.
Re:Security Alert: Ethereum Constantinople Postponement
by aakilfernandes on Tuesday January 15, 2019 @09:34PM
(#18917819)
There's a simple way to prevent re-entrency.
----------
bool isEntered = false;
function doSomething() { if (isEntered) { throw; } isEntered = true; ...; isEntered = false; }
----------
I don't really understand why they went with this gas-limit solution. I feel like there must be something I'm missing cause its too obvious.
Re:Security Alert: Ethereum Constantinople Postponement
by DennisP on Wednesday January 16, 2019 @06:37AM
(#18919961)
A similar method is to put the external call at the very end of your function, after any state updates, and make sure that function isn't called from another function. These are well-known techniques in the community and commonly used, which is probably one reason the researchers didn't find any vulnerabilities in deployed contracts.
The gas limit on send was more of a belt-and-suspenders thing, not intended as the sole protection; that may have been a bad idea in retrospect, but at the time people were pretty focused on adding every protection possible against another DAO situation.