The code grants maximum allowances (type(uint256).max) of linkToken and sdlToken to the _router.
This practice, while common for gas optimization, has proven to increase the impact of an attack many times in the past, in case the approved contract gets hacked.
Compromised routers can siphon unlimited funds.
Manual Analysis
Consider approving the exact amount that’s needed to be transferred, or alternatively, add an external function that allows the revocation of approvals.
The contest is live. Earn rewards by submitting a finding.
This is your time to appeal against judgements on your submissions.
Appeals are being carefully reviewed by our judges.