code-423n4 / 2022-05-sturdy-findings

7 stars 3 forks source link

Upgraded Q -> M from 135 [1654503723748] #173

Closed code423n4 closed 2 years ago

code423n4 commented 2 years ago

Judge has assessed an item in Issue #135 as Medium risk. The relevant finding follows:

HickupHH3 commented 2 years ago

"At YieldManager.sol, _convertAssetToExchangeToken() function converts the assets to exchange token at the limits of the slippage. However, there is no oracle provided and the AMM prices are subject to manipulation. The team might consider to add a function parameter for minimum tokens to receive."

Duplicate of #61