Platform: Code4rena
Start Date: 29/07/2022
Pot Size: $50,000 USDC
Total HM: 6
Participants: 75
Period: 5 days
Judge: GalloDaSballo
Total Solo HM: 3
Id: 149
League: ETH
Rank: 64/75
Findings: 1
Award: $56.12
🌟 Selected for report: 0
🚀 Solo Findings: 0
🌟 Selected for report: oyc_109
Also found by: 0x1f8b, 0x52, 0xNazgul, 0xSmartContract, 0xf15ers, 8olidity, Aymen0909, Bnke0x0, CertoraInc, Chom, CodingNameKiki, Deivitto, Dravee, ElKu, IllIllI, JC, Lambda, Noah3o6, NoamYakov, RedOneN, Respx, ReyAdmirado, Rohan16, Rolezn, Ruhum, Sm4rty, TomJ, Twpony, Waze, Yiko, __141345__, ajtra, apostle0x01, ashiq0x01, asutorufos, bardamu, benbaessler, berndartmueller, bharg4v, bulej93, c3phas, cccz, ch13fd357r0y3r, codexploder, cryptonue, cryptphi, defsec, djxploit, durianSausage, fatherOfBlocks, gogo, hansfriese, horsefacts, ignacio, kyteg, lucacez, mics, rbserver, robee, sashik_eth, simon135, sseefried, tofunmi, xiaoming90
56.1238 USDC - $56.12
Add Reentrancy Guard to Multiple Functions to Receiver Implementation Contract.
#0 - re1ro
2022-08-04T23:34:55Z
Invalid
ReceiverImplementation is used by the DepositReceiver which is deployed and destroyed immediately. There is nowhere to reenter
#1 - GalloDaSballo
2022-08-28T22:13:00Z
Because the finding points towards the breaking of CEI pattern I believe It's correct to leave as valid Low.
However the submission is really underwhelming and should be penalized
#2 - GalloDaSballo
2022-08-28T22:13:05Z
1L
#3 - re1ro
2022-09-15T01:51:09Z
Again DepositReceiver that is using ReceiverImplementation doesn't have any methods besides constructor. So there is literally nowhere to re-enter. Still invalid from our point of view