DeFiFoundrySolidity
16,653 OP
View results
Submission Details
Severity: low
Invalid

Redundant verification before swap

Summary

On StrategyArb.sol@L86 there is a balance verification before performing the swap that isn't necessary.

Vulnerability Details

The Transmutter::claim done before it makes sure there is a balance of the underlying token at least equal to the amount passed to the function making the balance verification unnecessary.

Impact

Extra gas cost and unnecessary code

Tools Used

Manual review

Recommendations

Remole line L85 and L86

Updates

Appeal created

inallhonesty Lead Judge 6 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
inallhonesty Lead Judge 6 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.