First Flight #12: Kitty Connect

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

New Cat NFT ID is not pushed in `s_ownerToCatsTokenId` mapping in `mintBridgedNFT`

Summary

New Cat NFT ID is not pushed in s_ownerToCatsTokenId mapping in mintBridgedNFT

Vulnerability Details

When a new cat NFT is minted in the mintBridgedNFT function, the new cat NFT ID is not pushed in the s_ownerToCatsTokenId mapping.

Impact

This can basically disrupts the functionallity of the protocol and breaks the interaction of this user with it since its list of NFTs is not updated.

Tools Used

Manual review

Recommendations

Consider adding this line in mintBridgedNFT function:

+ s_ownerToCatsTokenId[catOwner].push(tokenId);
Updates

Lead Judging Commences

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

owner's token ID array not updated in `mintBridgedNFT`

Support

FAQs

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