Submission Details
Severity:
`GMXCallback::afterDepositCancellation(), afterDepositExecution(),afterWithdrawalExecution()`
Summary
Reducing Storage Reads, _store is a storage variable that is read multiple times within the function, so storing the value in a local variable.
Vulnerability Details
In the code, every reference to vault.store() would result in a storage read operation, which consumes gas.
Impact
Wastes gas
Tools Used
Manual Review
Recommendations
Storing the value of vault.store() in a local variable _store as GMXTypes.Store storage ,
-GMXTypes.Store memory _store = vault.store();
+GMXTypes.Store storage _store = vault.store();
Prize pool breakdown
Total prize
35,000 USDC
nSLOC
2,28915 USDC / LOC
33,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.