No slippage protection in sellProfits() function?
This caught my attention, while wrapping up my audit of this contest, regarding parameter of ISwapRouter.ExactInputSingleParams({:
amountOutMinimum: 0,
From my limited understanding this is related to slippage, and if zero, there is no slippage protection...
Vulnerable to sandwich attacks?
Can swap and get out a lot less than expected...?
VSC, manual, overtiredness...
Add a suitable non-zero amountOutMinimum amount...
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.