We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
tensors
Not declaring a variable can save gas.
https://github.com/code-423n4/2021-09-bvecvx/blob/1d64bd58c7a4224cc330cef283561e90ae6a3cf5/veCVX/contracts/locker/CvxLocker.sol#L354
Since the locksLength variable isn't used often, you can save gas by omitting it.
Use locks.length instead.
The text was updated successfully, but these errors were encountered:
tensors issue #7
e79354c
No branches or pull requests
Handle
tensors
Vulnerability details
Impact
Not declaring a variable can save gas.
Proof of Concept
https://github.com/code-423n4/2021-09-bvecvx/blob/1d64bd58c7a4224cc330cef283561e90ae6a3cf5/veCVX/contracts/locker/CvxLocker.sol#L354
Since the locksLength variable isn't used often, you can save gas by omitting it.
Recommended Mitigation Steps
Use locks.length instead.
The text was updated successfully, but these errors were encountered: