Beginner FriendlyFoundryNFT
100 EXP
View results
Submission Details
Severity: medium
Valid

withdrawFees function can be DOS'd by sending ether to the contract

Summary

withdrawFees function can be DOS'd by sending ether to the contract.

Vulnerability Details

Don't rely on strict equality when comparing balances. If a malicious actor send 1 wei to the contract using selfdestruct the withdrawFees function is DOS'd.

Impact

Fee address can't withdraw fees anymore.

Tools Used

Manual review

Recommendations

Use this to check if there are active players.

require(players.length == 0, "PuppyRaffle: There are currently players active!");
Updates

Lead Judging Commences

Hamiltonite Lead Judge about 2 years ago
Submission Judgement Published
Validated
Assigned finding tags:

greifers-send-money-to-contract-to-block-withdrawfees

Support

FAQs

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

Give us feedback!