Beginner FriendlySolidity
100 EXP
View results
Submission Details
Severity: low
Valid

Precision Loss in Calculation within buyOutEstateNFT

Summary:
The calculation in InheritanceManager::buyOutEstateNFT may suffer from precision loss, particularly when dealing with ERC20 tokens with small decimal values.

Vulnerability Details:
Using integer division in the expression
uint256 finalAmount = (value / divisor) * multiplier;
can lead to rounding errors because Solidity rounds down on division.

Impact:
Low – Results in minor discrepancies that could be critical when dealing with precise financial calculations.

Tools Used:
slither, aderyn, foundry

Recommendations:
Either restrict the asset types to those with appropriate decimal precision, introduce precision values or adjust the calculation method to maintain precision.

Updates

Lead Judging Commences

0xtimefliez Lead Judge 10 months ago
Submission Judgement Published
Validated
Assigned finding tags:

truncation of integers

Support

FAQs

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

Give us feedback!