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

Reentrancy possible in `refund`

Summary

If a contract having a fallback/receive function invokes refund, a reentrancy is possible, which could lead to all funds being drained in PuppyRaffle.

Vulnerability Details

The attacker can intercept the funds and call back refund until all Ether in the contract are wiped out.

Impact

Systemic impact on the protocol and all its actors.

Tools Used

Recommendations

Use Checks Effects Interactions pattern in refund.

Updates

Lead Judging Commences

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

reentrancy-in-refund

reentrancy in refund() function

Support

FAQs

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