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 8 months ago
Submission Judgement Published
Invalidated
Reason: Too generic

Support

FAQs

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