Beginner FriendlyFoundryNFT
100 EXP
View results
Submission Details
Severity: low
Invalid

Contract lacks onlyOwner check causing anyone to call the selectWinner and withdrawFess function

Summary

The contract lacks onlyOwner checks on the selectWinner and withdrawFees functions.

Vulnerability Details

Though the docs don't mention that these functions should have onlyOwner checks but if thought logically the selectWinner and withdrawFees function should only be accessible to the owner of the contarct instead of anyone to have an access over these functions.

Impact

Anyone can access the selectWinner and withdrawFees functions.

Tools Used

VS Code

Recommendations

Add onlyOwner modifier to selectWinner and withdrawFees functions.

function selectWinner() external onlyOwner{
...
}
function withdrawFees() external onlyOwner{
...
}
Updates

Lead Judging Commences

Hamiltonite Lead Judge almost 2 years ago
Submission Judgement Published
Invalidated
Reason: User experience and design improvement

Support

FAQs

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