Platform: Code4rena
Start Date: 17/03/2022
Pot Size: $30,000 USDC
Total HM: 8
Participants: 43
Period: 3 days
Judge: gzeon
Total Solo HM: 5
Id: 100
League: ETH
Rank: 31/43
Findings: 1
Award: $60.11
🌟 Selected for report: 0
🚀 Solo Findings: 0
🌟 Selected for report: defsec
Also found by: 0x1f8b, 0xDjango, 0xNazgul, 0xkatana, 0xwags, CertoraInc, Funen, GeekyLumberjack, GreyArt, IllIllI, Kenshin, Ruhum, TerrierLover, WatchPug, berndartmueller, bugwriter001, cccz, cmichel, csanuragjain, hake, kenta, kirk-baird, leastwood, minhquanym, oyc_109, peritoflores, rayn, remora, rfa, robee, saian, samruna, sorrynotsorry, wuwe1
60.1124 USDC - $60.11
Collateral.sol:169 require(_amountWithdrawn > _fee, "Withdrawal amount too small"); Statement should be corrected to require(_amountWithdrawn >= _fee, "Withdrawal amount too small");
Collateral.sol:191 Zero address check is not performed. Statement can be added as; require(_newStrategyController != address(0), "Error msg here")
Collateral.sol:229 Zero address check is not performed.Statement can be added as; require(_newDepositHook != address(0), "Error msg here")
Collateral.sol:238 Zero address check is not performed.Statement can be added as; require(_newWithdrawHook != address(0), "Error msg here")
CollateralDepositRecord.sol:20 _newGlobalDepositCap var is not checked being zero. Statement can be added as; require(_newGlobalDepositCap != 0, "Error msg here")
CollateralDepositRecord.sol:21 _newAccountDepositCap var is not checked being zero. Statement can be added as; require(_newAccountDepositCap != 0, "Error msg here")
WithdrawHook.sol:30 _newVault var is not checked being zero. Statement can be added as; require(_newVault != address(0), "Error msg here")
#0 - ramenforbreakfast
2022-03-23T01:24:33Z
1 is duplicate of #48 2-4 are zero address duplicates #35 5-7 are cap and vault zero value issues, duplicates #35