DeFiFoundry
20,000 USDC
View results
Submission Details
Severity: high
Invalid

Lack of Access Control on 'auctionEnd'

Summary

There's no access control mechanism preventing anyone from ending the auction or claiming tokens. Only the owner can end the auction, but anyone can claim tokens after the auction ends.

Vulnerability Details

Impact

Any external user can call the auctionEnd function to finalize the auction.

Tools Used

Recommendations

Mitigation: Add a modifier that restricts this function to the owner, or carefully review the potential implications of allowing anyone to call it.

modifier onlyOwner() {
require(msg.sender == owner, "Not owner");
_;
}
function auctionEnd() external onlyOwner {
// Logic...
}
Updates

Lead Judging Commences

inallhonesty Lead Judge 12 months ago
Submission Judgement Published
Invalidated
Reason: Too generic

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.