Upgraded Q -> M from #351 [1674418158719] #667
Labels
2 (Med Risk)
Assets not at direct risk, but function/availability of the protocol could be impacted or leak value
duplicate-533
satisfactory
satisfies C4 submission criteria; eligible for awards
Judge has assessed an item in Issue #351 as M risk. The relevant finding follows:
Stable Vault cannot accept tokens with more then 18 decimals because of this lines:
https://github.com/code-423n4/2022-12-tigris/blob/588c84b7bb354d20cbca6034544c4faa46e6a80e/contracts/StableVault.sol#L49
https://github.com/code-423n4/2022-12-tigris/blob/588c84b7bb354d20cbca6034544c4faa46e6a80e/contracts/StableVault.sol#L67
Consider using different conversion formula. For example: amount * 1e18 / 10token.decimals() for deposit and amount * 10token.decimals() / 1e18 for withdraw
The text was updated successfully, but these errors were encountered: