Submission Details
Severity:
Accessing storage variables already instantiated in memory
Summary
Accessing storage instead of memory on lines 262 and 263 in Lender.sol incurs extra gas costs
Vulnerability Details
pools[poolId] on lines 262 and 263 are accessing storage when they can instead access memory; the memory variable pool has been set equal to pools[poolId] in the function.
Impact
Replacing 2 warm SLOADs (100 gas) with 2 MLOADs (3 gas) saves 194 gas here. Additionally, the gas cost of reading from an array is avoided.
Tools Used
Recommendations
Change pools[poolId] to pool on lines 262 and 263
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.