Platform: Code4rena
Start Date: 05/05/2022
Pot Size: $125,000 DAI
Total HM: 17
Participants: 62
Period: 14 days
Judge: leastwood
Total Solo HM: 15
Id: 120
League: ETH
Rank: 45/62
Findings: 1
Award: $201.23
🌟 Selected for report: 0
🚀 Solo Findings: 0
🌟 Selected for report: IllIllI
Also found by: 0v3rf10w, 0x1f8b, 0x4non, 0xDjango, 0xNazgul, 0xf15ers, 0xkatana, 0xsomeone, AlleyCat, BowTiedWardens, Cityscape, Fitraldys, Funen, GimelSec, Hawkeye, JC, MaratCerby, MiloTruck, Randyyy, TerrierLover, Tomio, UnusualTurtle, WatchPug, Waze, _Adam, augustg, bobirichman, catchup, csanuragjain, ellahi, fatherOfBlocks, hake, hansfriese, horsefacts, ignacio, joestakey, kenta, mics, oyc_109, robee, samruna, sashik_eth, sikorico, simon135, throttle
201.2269 DAI - $201.23
Remove the use of Reentrancy Guard.. Nonreentrant takes 4 SLOADS or about 3500 gas per call. You save a ton of gas over time by just using proper checks/effects/interactions patterns(which you're already doing).
The openzeppelin reentrancy guard is in my opinion, an expensive way to handle the issue. You will save about 1m in gas every 250~ times a function is called by removing nonreentrant and fixing any checks/effects/interactions patterns that may need fixing.
#0 - 0xfoobar
2022-05-30T07:10:04Z
Good callout