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

In the setRouter function, directly setting a new router and approving the underlying token is risky.

Summary

Vulnerability Details

In the setRouter function, directly setting a new router and approving the underlying token is risky. It is recommended to first set the approval amount of the previous router to 0 before changing the router address. This precaution helps prevent potential approval-related attacks by ensuring that no residual allowance is left for the old router, which might be exploited.


Impact

Tools Used

Recommendations

Updates

Appeal created

inallhonesty Lead Judge 11 months ago
Submission Judgement Published
Invalidated
Reason: Too generic

Support

FAQs

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

Give us feedback!