forks/go-ethereum
1
0
Fork 1
mirror of https://github.com/ethereum/go-ethereum.git synced 2026-07-26 06:36:43 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Prevent re-entrancy, fix balance check logic

Browse source
This commit is contained in:
Oleksii Matiiasevych 2016-10-26 12:41:56 +03:00 committed by GitHub
parent 4936bec6c9
commit ee24dffae2
1 changed files with 6 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
swarm/services/chequebook/contract/chequebook.sol
View file

@ -27,10 +27,12 @@ contract chequebook is mortal {
if(owner != ecrecover(hash, sig_v, sig_r, sig_s)) return;
// Attempt sending the difference between the cumulative amount on the cheque
// and the cumulative amount on the last cashed cheque to beneficiary.
if (amount - sent[beneficiary] >= this.balance) {
if (beneficiary.send(amount - sent[beneficiary])) {
// Upon success, update the cumulative amount.
if (amount - sent[beneficiary] <= this.balance) {
// Update the cumulative amount upfront to prevent re-entrancy.
sent[beneficiary] = amount;
if (!beneficiary.send(amount - sent[beneficiary])) {
// Upon failure, rollback.
throw;
}
} else {
// Upon failure, punish owner for writing a bounced cheque.