Platform: Code4rena
Start Date: 08/11/2022
Pot Size: $60,500 USDC
Total HM: 6
Participants: 72
Period: 5 days
Judge: Picodes
Total Solo HM: 2
Id: 178
League: ETH
Rank: 64/72
Findings: 1
Award: $36.34
🌟 Selected for report: 0
🚀 Solo Findings: 0
🌟 Selected for report: RaymondFam
Also found by: 0x1f8b, 0x52, 0xSmartContract, 0xc0ffEE, 0xhacksmithh, 8olidity, Awesome, BClabs, Bnke0x0, Chom, Deivitto, Hashlock, IllIllI, Josiah, KingNFT, Nyx, R2, ReyAdmirado, Rolezn, SamGMK, Sathish9098, SinceJuly, V_B, Vadis, Waze, a12jmx, adriro, ajtra, aphak5010, bearonbike, bin, brgltd, carlitox477, carrotsmuggler, cccz, ch0bu, chaduke, datapunk, delfin454000, erictee, fatherOfBlocks, fs0c, horsefacts, jayphbee, ktg, ladboy233, pashov, perseverancesuccess, rbserver, ret2basic, tnevler, zaskoh
36.3434 USDC - $36.34
It is recommended to use safeTransferFrom() instead of transferFrom() when transferring ERC721. The recipient could have logic in the onERC721Received() function, which is only triggered in the safeTransferFrom() function and not in transferFrom(). There is the potential loss of NFTs If the recipient is unable to handle the sent ERC721s.
Manuel Review
Use safeTransferFrom() when sending out the NFTs.
#0 - c4-judge
2022-11-21T15:18:14Z
Picodes marked the issue as duplicate of #174
#1 - c4-judge
2022-12-11T16:35:03Z
Picodes marked the issue as not a duplicate
#2 - c4-judge
2022-12-11T16:35:15Z
Picodes changed the severity to QA (Quality Assurance)
#3 - c4-judge
2022-12-11T16:35:47Z
Picodes marked the issue as grade-b