Platform: Code4rena
Start Date: 20/09/2022
Pot Size: $100,000 USDC
Total HM: 4
Participants: 109
Period: 7 days
Judge: GalloDaSballo
Id: 163
League: ETH
Rank: 94/109
Findings: 1
Award: $55.20
🌟 Selected for report: 0
🚀 Solo Findings: 0
🌟 Selected for report: IllIllI
Also found by: 0x1f8b, 0x4non, 0x52, 0x5rings, 0xNazgul, 0xRobocop, 0xSmartContract, 0xdeadbeef, 0xsanson, 8olidity, Amithuddar, Aymen0909, B2, B353N, CertoraInc, Ch_301, Chom, CodingNameKiki, Deivitto, ElKu, Funen, JC, JohnnyTime, Kresh, Lambda, Noah3o6, RaymondFam, ReyAdmirado, RockingMiles, Rolezn, Sm4rty, SuldaanBeegsi, Tadashi, TomJ, Tomio, V_B, Waze, __141345__, a12jmx, ak1, arcoun, asutorufos, aviggiano, berndartmueller, bharg4v, bin2chen, brgltd, bulej93, c3phas, catchup, cccz, ch0bu, cryptonue, cryptphi, csanuragjain, delfin454000, devtooligan, djxploit, durianSausage, eighty, erictee, exd0tpy, fatherOfBlocks, giovannidisiena, hansfriese, ignacio, joestakey, ladboy233, lukris02, m9800, malinariy, martin, minhtrng, obront, oyc_109, pedr02b2, pedroais, pfapostol, philogy, prasantgupta52, rbserver, ronnyx2017, rotcivegaf, rvierdiiev, sach1r0, shung, simon135, throttle, tnevler, tonisives, wagmi, yixxas, zkhorse, zzykxx, zzzitron
55.1985 USDC - $55.20
_safeMint() should be used rather than _mint() wherever possibleThere are 3 instances of this issue:
File: src/ArtGobblers.sol 356: _mint(msg.sender, gobblerId); 389: _mint(msg.sender, gobblerId); 469: _mint(msg.sender, gobblerId);
https://github.com/code-423n4/2022-09-artgobblers/blob/main/src/ArtGobblers.sol
https://github.com/code-423n4/2022-09-artgobblers/blob/main/src/ArtGobblers.sol
Each event should use three indexed fields if there are three or more fields
There are 1 instances of this issue:
File: src/ArtGobblers.sol 236: event RandomnessFulfilled(uint256 randomness);
https://github.com/code-423n4/2022-09-artgobblers/blob/main/src/ArtGobblers.sol
#0 - GalloDaSballo
2022-10-14T00:36:46Z
1L
Even is misguided