First Flight #12: Kitty Connect

First Flight #12: Kitty Connect
Beginner FriendlyFoundryNFTGameFi
100 EXP
View results
Submission Details
Severity: low
Valid

[L1] getCatAge(uint256 tokenId) could overflow

Summary

There no check when setting Cat age , so the function getCatAge(uint256 tokenId) could overflow if the admin set an age after block.timestamp

Vulnerability Details

There no check when setting Cat age , so the function getCatAge(uint256 tokenId) could overflow if the admin set an age after block.timestamp

Impact

Overflow

Tools Used

manual

Recommendations

check if the age is before block.timestamp before recording Cat info

Updates

Lead Judging Commences

inallhonesty Lead Judge about 1 year ago
Submission Judgement Published
Validated
Assigned finding tags:

Inputed cat dob can be in the future, making a function revert due to underflow.

Support

FAQs

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