Merge pull request #3242 from ethereum/chriseth-patch-1

Bugfix in blind auction
This commit is contained in:
Alex Beregszaszi
2017-11-24 13:50:01 +00:00
committed by GitHub
+3 -4
View File
@@ -492,7 +492,7 @@ high or low invalid bids.
if (amount > 0) {
// It is important to set this to zero because the recipient
// can call this function again as part of the receiving call
// before `send` returns (see the remark above about
// before `transfer` returns (see the remark above about
// conditions -> effects -> interaction).
pendingReturns[msg.sender] = 0;
@@ -508,12 +508,11 @@ high or low invalid bids.
require(!ended);
AuctionEnded(highestBidder, highestBid);
ended = true;
// We send all the money we have, because some
// of the refunds might have failed.
beneficiary.transfer(this.balance);
beneficiary.transfer(highestBid);
}
}
.. index:: purchase, remote purchase, escrow
********************