Alchemix Transmuter
Alchemix
DeFiFoundrySolidity
16,653 OP
View results
Previous Next
Submission Details
Severity: low
Invalid

Function `_harvestAndReport` has unused variables `claimable`.

0x6a70

Summary

In all of the three contracts there is a call to the `transmuter` to get the claimable balance of the strategy:

https://github.com/Cyfrin/2024-12-alchemix/blob/82798f4891e41959eef866bd1d4cb44fc1e26439/src/StrategyMainnet.sol#L178

https://github.com/Cyfrin/2024-12-alchemix/blob/82798f4891e41959eef866bd1d4cb44fc1e26439/src/StrategyArb.sol#L154

https://github.com/Cyfrin/2024-12-alchemix/blob/82798f4891e41959eef866bd1d4cb44fc1e26439/src/StrategyOp.sol#L167

This uint256 `claimable` variable that is initialized is never used!

Vulnerability Details

Calling for function and initializing a variable that is never used.

Impact

Unnecessary usage of gas.

Tools Used

Manual review.

Recommendations

Discard that peace of code.

Updates

Appeal created

inallhonesty Lead Judge 7 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.