Submission Details
Severity:
use `_safeMint()` instead of `_mint()`
Summary
use _safeMint() instead of _mint()
Vulnerability Details
The usage of _safeMintguarantees that the receiver toaddress is either a smart contract that implements IERC721Receiver.onERC721Receivedor an EOA.
Impact
Using ERC721::_mint() can mint ERC721 tokens to addresses which don't support ERC721 tokens, while ERC721::_safeMint() ensures that ERC721 tokens are only minted to addresses which support them. OpenZeppelin discourages the use of _mint().
Tools Used
Recommendations
Use _safeMint() instead of _mint() for ERC721.
Updates
Lead Judging Commences
inallhonesty about 1 year ago
Submission Judgement Published Reason: Known issue
Assigned finding tags:
Known - LightChaser
https://github.com/Cyfrin/2024-05-Sablier/issues/1
Prize pool breakdown
Total prize
53,440 USDC
nSLOC
2,65520 USDC / LOC
45,000 USDC
3,100 USDC
5,340 USDC
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.