Sablier

Sablier
DeFiFoundry
53,440 USDC
View results
Submission Details
Severity: low
Invalid

Use _safemint instead of _mint for ERC721

Summary

Use safemint instead of _mint for ERC721

Vulnerability Details

Whenever a stream is created, the receipient of the stream is minted an NFT. The issue here is that some contracts does not support ERC721 and given such a cases, the minted NFT will be frozen in the contract and cannot be used.

Impact

Users/ Recipients will lose their stream NFTs. (and if need arises for them to sell they cant be able to do so)

Tools Used

Manual Review

Recommendations

Use safemint instead of mint to assert if the recipient contract can handle the NFTs.

Updates

Lead Judging Commences

inallhonesty Lead Judge about 1 year ago
Submission Judgement Published
Invalidated
Reason: Known issue
Assigned finding tags:

Known - LightChaser

https://github.com/Cyfrin/2024-05-Sablier/issues/1

Support

FAQs

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