DeFiFoundry
50,000 USDC
View results
Submission Details
Severity: low
Invalid

Rounding errors in shares

Description:

Small deposits near minDepositAmount (1000 USDC) may result in zero shares due to truncation in division (shares = amount * totalShares / totalAmount).

Impact:

  • Depositors lose funds if shares round to zero.

  • Violates Depositor Share Value Preservation.

Affected Code:

  • PerpetualVault.sol#_mint() calculates shares with integer division.

Recommendation:
Enforce a minimum share precision (e.g., 1e6) or use a shares model with higher decimals.

Updates

Lead Judging Commences

n0kto Lead Judge 5 months ago
Submission Judgement Published
Invalidated
Reason: Too generic
Assigned finding tags:

Suppositions

There is no real proof, concrete root cause, specific impact, or enough details in those submissions. Examples include: "It could happen" without specifying when, "If this impossible case happens," "Unexpected behavior," etc. Make a Proof of Concept (PoC) using external functions and realistic parameters. Do not test only the internal function where you think you found something.

Support

FAQs

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