The Standard

The Standard
DeFiHardhat
20,000 USDC
View results
Submission Details
Severity: low
Invalid

Incompatibility with > 18 decimals assets

Summary

Incompatibility with > 18 decimals assets

Vulnerability Details

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.

Impact

Assets that have > 18 decimals cannot be used.

Tools Used

Manual review

Recommendations

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

Updates

Lead Judging Commences

hrishibhat Lead Judge over 1 year ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
Assigned finding tags:

informational/invalid

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.