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

WITHDRAW_FEES_BLOCKAGE_BY_ETH_SENDING

Summary

Withdraw is blocked if user sent ether to the contract

Vulnerability Details

In withdrawFees function in require(address(this).balance == uint256(totalFees), "PuppyRaffle: There are currently players active!"); is checked that balance of the contract equals to the counter value. If someone sent eth to the contract, withdraw is blocked

Impact

Withdraw fees blockage

Tools Used

Recommendations

Do not compare contract balance with counter

Updates

Lead Judging Commences

Hamiltonite Lead Judge almost 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.