Submission Details
Severity:
Use unchecked for collateral
Summary
Use unchecked when dividing can save gas.
Vulnerability Details
In lender.sol,the borrow() function has,
if (collateral == 0) revert ZeroCollateral();
// make sure the user isn't borrowing too much
uint256 loanRatio = (debt * 10 ** 18) / collateral;
We can use unchecked for loanRatio calculation since collateral will never be zero because of previous statement.
Impact
Higher runtime gas usage
Tools Used
Manual review
Recommendations
consider using
if (collateral == 0) revert ZeroCollateral();
// make sure the user isn't borrowing too much
unchecked{
uint256 loanRatio = (debt * 10 ** 18) / collateral;
}
Prize pool breakdown
Total prize
20,000 USDC
nSLOC
70628 USDC / LOC
18,000 USDC
2,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.