Submission Details
Severity:
Emitting cached values instead of storage variables saves gas
Summary
Caching of a state variable replaces each Gwarmaccess (100 gas) with a much cheaper stack read.
Total Instances: 4
Estimated Gas Saved: 4 * 100 = 400
Recommendations
use already calculated totalDebt and cached loan variables to save 2SLOAD
emit Borrowed(
loan.borrower,
pool.lender,
loanId,
- loans[loanId].debt, // use totalDebt here
- loans[loanId].collateral, // loan.collateral here
+ totalDebt,
+ loan.collateral,
pool.interestRate,
block.timestamp
);
Tools used
manual code review
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.