In SmartVaultV3.sol
the Uniswap Fee is hardcoded to 3000, which will lead to non optimal routes. This will lead to losses compared to optimal routing
This hardcoded value will lead to using pools with higher fees. As stated in the Uniswap Docs:
" fee
The fee tier of the pool, used to determine the correct pool contract in which to execute the swap"
For example the there is aLINK/USDC
pool for 0.05% fee, as well as ARB / USDC
for 0.05%` fee and so on.
So the optimal route is different than 0.3% as hardcoded.
Therefore using the current implementation would create a significant loss of revenue.
Manual review
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.