Platform: Code4rena
Start Date: 18/10/2022
Pot Size: $50,000 USDC
Total HM: 13
Participants: 67
Period: 5 days
Judge: Picodes
Total Solo HM: 7
Id: 172
League: ETH
Rank: 60/67
Findings: 1
Award: $25.96
🌟 Selected for report: 0
🚀 Solo Findings: 0
🌟 Selected for report: Jeiwan
Also found by: 0x1f8b, 0x4non, 0x5rings, 0xSmartContract, Awesome, Aymen0909, Bnke0x0, CodingNameKiki, Diana, DimSon, JC, JrNet, LeoS, RaymondFam, ReyAdmirado, Saintcode_, Shinchan, __141345__, berndartmueller, bharg4v, brgltd, carlitox477, ch0bu, chaduke, cryptostellar5, emrekocak, gogo, lukris02, martin, mcwildy, sakman, trustindistrust, zishansami
25.9629 USDC - $25.96
_safemint()
should be used rather than _mint()
wherever possibleThere are 4 instances of this issue.
File: contracts/JBTiered721Delegate.sol - line 461
File: contracts/JBTiered721Delegate.sol - line 504
File: contracts/JBTiered721Delegate.sol - line 635
File: contracts/JBTiered721Delegate.sol - line 677
_mint()
is discouraged in favor of _safeMint()
which ensures that the recipient is either an EOA or implements IERC721Receiver
.
#0 - c4-judge
2022-11-08T17:46:51Z
Picodes marked the issue as grade-b
#1 - c3phas
2022-12-03T19:29:03Z
Shouldn't this be under QA