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

The token can be lost due to mint to the NonERC721Receiver.

Summary

The token can be minted to a contract that does not have the functionality to transfer ERC721 tokens and token will be lost.

Vulnerability Details

The contract uses the _mint function to mint tokens for the raffle winner in the fulfillRandomWords function. This function doesn't have _check_on_erc721_received check and tokens may be lost if the receiver is contract and doesn't have methods for transferring ERC721 tokens.

Impact

High

Tools Used

VS Code, Manual analysis.

Recommendations

Consider using _safe_mint instead of _mint.

Updates

Lead Judging Commences

inallhonesty Lead Judge over 1 year ago
Submission Judgement Published
Invalidated
Reason: Incorrect statement

Support

FAQs

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