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

Low issues

[1] StrategyArb::setRouter does not remove previous approvals

When new router is set using setRouter function, the approval to old router address is not removed. Since the protocol sets infinite approvals, there should be a way remove approval in case of emergency or simply when the router will no longer be used.

[2] StrategyOp::setRouter does not remove previous approvals

When new router is set using setRouter function, the approval to old router address is not removed. Since the protocol sets infinite approvals, there should be a way remove approval in case of emergency or simply when the router will no longer be used.

Updates

Appeal created

inallhonesty Lead Judge 10 months ago
Submission Judgement Published
Invalidated
Reason: Lack of quality

Support

FAQs

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