-
-
Notifications
You must be signed in to change notification settings - Fork 15
Open
Description
In AllPayAuction.sol and EnglishAuction.sol, the withdraw functions are external but do not verify that the msg.sender is the auctioneer.
Issue: Anyone can trigger the withdrawal of funds to the auctioneer's address.
Risk: While the funds go to the correct person (the auctioneer), this allows external parties to force financial realizations for the auctioneer, which might have tax or accounting implications, or interfere with a manager's planned strategy.
Fix: Add require(msg.sender == auction.auctioneer) to the withdraw functions.
@kaneki003 Please check this out , /assign
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels