Submission Details
Severity:
[GAS-3] Use `storage` instead of `memory` for structs/arrays
Summary
Using memory copies the struct or array in memory. Use storage to save the location in storage and have cheaper reads:
Vulnerability Details
Instances (12):
File: src/Fees.sol
30: ISwapRouter.ExactInputSingleParams memory params = ISwapRouter
File: src/Lender.sol
117: Loan memory loan = loans[loanId];
238: Pool memory pool = pools[poolId];
249: Loan memory loan = Loan({
296: Loan memory loan = loans[loanId];
363: Loan memory loan = loans[loanId];
367: Pool memory pool = pools[poolId];
441: Loan memory loan = loans[loanId];
467: Loan memory loan = loans[loanId];
552: Loan memory loan = loans[loanId];
606: Loan memory loan = loans[loanId];
611: Pool memory pool = pools[poolId];
Impact
Tools Used
Recommendations
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.