contracts icon indicating copy to clipboard operation
contracts copied to clipboard

Transfer() may prevent wallet to interact with non updated fallbacks after the Istantbul fork

Open i-stam opened this issue 5 years ago • 3 comments

Regular call() should be used instead, Check also https://diligence.consensys.net/blog/2019/09/stop-using-soliditys-transfer-now/ .

There should be carefully examined it there are bugs introduced by re-entrancy. It should be possibly combined with restricting owner to EOAs (check #570)

i-stam avatar Apr 24 '20 11:04 i-stam

This is fixed in v4. We have circumvented this issue by using executeTransaction() in the backend.

i-stam avatar May 02 '20 12:05 i-stam

@nostdm i think you should go in and understand this stuff too ...

mischat avatar May 06 '20 15:05 mischat

@mischat can you look at this too

mischat avatar May 06 '20 15:05 mischat