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

Theft of funds by reentrancy

Summary

Reentrancy allows for excessive refunds that deplete the contract's assets

Vulnerability Details

The refund function returns the entranceFee to the player. However, by re-entrancy when transferring the money, the entranceFee can be claimed repeatedly.

Impact

Project loses all funding.

Tools Used

Manual

Recommendations

Utilize the nonReentrancy modifier. Update the state according to the check-effect interaction pattern before remitting.

Updates

Lead Judging Commences

Hamiltonite Lead Judge over 1 year 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.