Submission Details
Severity:
Cache Storage Values && Unnecessary initialisation to default values
Summary
When accessing same storage value (
.length) multiple times, it is better to cache the value to save the gas, as reading from storage charges more gas.
function getAccountCollateralValue(address user) public view returns (uint256 totalCollateralValueInUsd) {
for (uint256 i = 0; i < s_collateralTokens.length; i++) {
...
}
...
}
Unnecessary initialisation of the value
i = 0in loops.
constructor(address[] memory tokenAddresses, address[] memory priceFeedAddresses, address dscAddress) {
...
// For example ETH / USD, BTC / USD, MKR / USD, etc
for (uint256 i = 0; i < tokenAddresses.length; i++) {
...
}
...
}
Prize pool breakdown
Total prize
15,000 USDC
nSLOC
23664
USDC / LOC
14,000 USDC
1,000 USDC
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.