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

Potential loss of owners fees

Summary

Fees might get overflowed at some point

Vulnerability Details

Since casting is performed and fees are stored in a uint64 variable called totalFees, as opposed to uint256, fees might overflow and users might never be able to select a winner until owner withdraws their cut.

Impact

Low, but just due to likelyhood.

Tools Used

Manual inspection.

Recommendations

Set fees to uint256 to prolongue time before contract overflows.

Updates

Lead Judging Commences

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

unsafe cast of fee to uint64

Support

FAQs

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