Incompatibility with > 18 decimals assets
The current design of the LiquidityPool limits its compatibility to only those tokens that have a decimal count of 18 or fewer. If an attempt is made to use a token that have more than 18 decimals, distributeAssets()
will throw due to an an underflow error during the costInEuros
calculation. This poses no financial risk but restricts the contract's adaptability in the wider DeFi ecosystem, preventing its use with tokens that have more than 18 decimals.
Assets that have > 18 decimals cannot be used.
Manual review
Adjust the decimals computation to handle both situations where token decimals might be greater or less than 18, Here's a simple implementation in Solidity: https://twitter.com/CharlesWangP/status/1742512078378725658
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.