Sablier

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

There are no checks to ensure `MAX_SEGMENT_COUNT <= 300`

Summary

There are no checks to ensure MAX_SEGMENT_COUNT <= 300

Vulnerability Details

In the documentation it is written that the max segments should be 300. There is an immutable variable that is used to ensure that the maximum number of segments is not exceeded. MAX_SEGMENT_COUNT value is set in the constructor but there are not checks if it is <= 300. It is an immutable variable meaning if it is set once it can not be changed so provide checks before setting it.

Impact

There are no checks to ensure MAX_SEGMENT_COUNT <= 300

Tools Used

Manual Review

Recommendations

If check to ensure the value is <= 300

Updates

Lead Judging Commences

inallhonesty Lead Judge
over 1 year ago
inallhonesty Lead Judge over 1 year ago
Submission Judgement Published
Invalidated
Reason: Incorrect statement

Support

FAQs

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