CurveModule: Redundant jToken #39
Labels
bug
Something isn't working
G (Gas Optimization)
sponsor acknowledged
Technically the issue is correct, but we're not going to resolve it for XYZ reasons
Handle
hickuphh3
Vulnerability details
Impact
jToken
is unused within the function, and therefore can be removed. While it can used to check againstICurvePool(curvePool).coins(j)
to ensure that the user is swapping for the right token, it isn't necessary because the final token balance is checked after all trades are executed.Recommended Mitigation Steps
Remove the
jToken
input parameter.The text was updated successfully, but these errors were encountered: