Submission Details
Severity:
Division before multiplication leads to minor precision loss
Summary
Division before multiplication leads to minor, yet still unnecessary precision loss.
Vulnerability Details
amountOutX18 = usdAmountInX18.div(indexPriceX18).mul(premiumDiscountFactorX18);
This instance of division before multiplication will lead to minor precision loss, and although minor due to using a fixed point math library, it could still cause dust funds to be stuck in the contract so funds could be lost.
Impact
Dust funds stuck in the contract.
Tools Used
Manual review
Recommendations
Multiply before divide to reduce unneeded precision loss.
Updates
Prize pool breakdown
Total prize
70,000 USDC
nSLOC
3,25821 USDC / LOC
66,500
3,500
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.