Sablier

Sablier
DeFiFoundry
53,440 USDC
View results
Submission Details
Severity: low
Invalid

Use scientific notation for large literal values

Summary

L-7: Large literal values multiples of 10000 can be replaced with scientific notation

Vulnerability Details

  • Found in src/SablierV2NFTDescriptor.sol Line: 200

    return streamedAmount * 10_000 / depositedAmount;
  • Found in src/libraries/SVGElements.sol Line: 217

    (10_000 - progressNumerical).toString(),

Impact

Low

Tools Used

Aderyn

Recommendations

Use e notation, for example: 1e18, instead of its full numeric value.

Updates

Lead Judging Commences

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

Info/Gas/Invalid as per Docs

https://docs.codehawks.com/hawks-auditors/how-to-determine-a-finding-validity

Support

FAQs

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