Platform: Code4rena
Start Date: 28/04/2022
Pot Size: $50,000 USDC
Total HM: 7
Participants: 43
Period: 5 days
Judge: gzeon
Total Solo HM: 2
Id: 115
League: ETH
Rank: 41/43
Findings: 1
Award: $59.06
🌟 Selected for report: 0
🚀 Solo Findings: 0
59.0559 USDC - $59.06
no need of using safemath for solidity 0.8+, saving lot of gas SuperVault.sol
prefer prefix ++i instead of suffix, results in gas savings DexAddressProvider.sol AdiminInceptionVault.sol
prefer uint256 instead of other uint types AdiminInceptionVault.sol AdiminInceptionVault.sol
break && in require statement into seperate require to save gas PARMinerV2.sol GenericMinerV2.sol
https://github.com/code-423n4/2022-04-mimo/blob/b18670f44d595483df2c0f76d1c57a7bfbfbc083/core/contracts/inception/InceptionVaultsCore.sol#L199
in function liquidatePartial() and borrow(), vualts _adminInceptionVault, _inceptionVaultsData and _VaultConfig can be gas-golfed via caching into stack, resulting in memory load instead of storage load, and storage is expensive. So, this results in lot of gas savings. They are called more than 2 times, so 2 sload savings.
lack of zero checks AdiminInceptionVault.sol