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

uint64 will cause overflow for totalFees

Summary

the contract keeps totalFees as uint64 which has a maximum limit of around 9.2 currency. Which is very low on side blockchains such as polygon. If the fees goes over it causes overflow losing the fee.

Vulnerability Details

uint64 is not enough to keep the fees as it will cause overflowing

Recommendations

Use uint128 to keep totalFees

Updates

Lead Judging Commences

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