Platform: Code4rena
Start Date: 28/06/2022
Pot Size: $25,000 USDC
Total HM: 14
Participants: 50
Period: 4 days
Judge: GalloDaSballo
Total Solo HM: 7
Id: 141
League: ETH
Rank: 36/50
Findings: 1
Award: $43.47
🌟 Selected for report: 0
🚀 Solo Findings: 0
🌟 Selected for report: zzzitron
Also found by: 0v3rf10w, 0x1f8b, 0x29A, AlleyCat, Bnke0x0, Chom, Funen, JC, Lambda, Limbooo, Meera, Picodes, Sm4rty, TerrierLover, TomJ, __141345__, asutorufos, aysha, c3phas, cccz, defsec, fatherOfBlocks, grGred, hake, ignacio, ladboy233, mrpathfindr, oyc_109, rfa, sach1r0, samruna, slywaters, ynnad
43.4708 USDC - $43.47
It is important for critical functionalities to add zero-address check. Lack of zero-address check in the _setPendingAdmin function of GovernorBravoDelegate.sol may lead to loss of control to admin functionalities in the event of accidentally setting the parameter to address(0).
I suggest adding this require statement:
require(newPendingAdmin != address(0));
#0 - GalloDaSballo
2022-08-13T23:36:23Z
Valid Low