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

Inconsistent router configuration functions across strategies

Summary

The StrategyMainnet contract lacks a setRouter() function that is present in both StrategyArb and StrategyOp contracts. This inconsistency in the strategy contracts, would make it not possible to change the router address for the StrategyMainnet contract.`

Recommendation

Consider implementing a consistent interface across all strategy contracts by adding a setRouter() function to StrategyMainnet if router configuration is needed, or document why this function is intentionally omitted. If the router configuration is not required for Mainnet, consider adding a comment explaining this design decision.

Updates

Appeal created

inallhonesty Lead Judge 6 months ago
Submission Judgement Published
Validated
Assigned finding tags:

Cannot Set A New Router In `StrategyMainnet.sol`

Support

FAQs

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