Timeline for Transfer function not working in basic deposit and withdraw contract
Current License: CC BY-SA 4.0
8 events
| when toggle format | what | by | license | comment | |
|---|---|---|---|---|---|
| Jan 21, 2019 at 15:01 | history | bumped | CommunityBot | This question has answers that may be good or bad; the system has marked it active so that they can be reviewed. | |
| Aug 13, 2018 at 14:38 | answer | added | HelloGaro | timeline score: 1 | |
| Aug 13, 2018 at 14:34 | comment | added | HelloGaro | Oh. it works on Ropsten. Guess it's a ganache problem? Thanks for the headsup about the reentrancy attacks. I will mark answered when I figure out why ganache is having problems with this. Thanks Jaime! | |
| Aug 13, 2018 at 14:27 | comment | added | Jaime | I sent one ether and I withdraw one ether. Just be sure that you are withdrawing with the same address that you deposited. Do you have this in a testnet (ropsten maybe?) If so, please post here the address of your deployed contract | |
| Aug 13, 2018 at 14:24 | comment | added | HelloGaro | Perhaps i don't know how to use remix properly? You had the proper value in the withdraw function? I am using ganache-cli now. I'll try the code out again with the test nets | |
| Aug 13, 2018 at 14:16 | comment | added | Jaime | I just ran this in remix and it works as intended. Btw you have a vulnerability to reentrancy attacks in your Withdraw function. | |
| Aug 13, 2018 at 14:05 | review | First posts | |||
| Aug 13, 2018 at 15:38 | |||||
| Aug 13, 2018 at 14:05 | history | asked | HelloGaro | CC BY-SA 4.0 |