Sablier

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

Missing checks can result in funds being locked up

Summary

When SablierV2MerkleLL was created, streamDurations were not checked and cliff>total was possible. If so, the claim function will revert and the funds will be locked.

Vulnerability Details

When cliff>total, the claim function will revert due to checking when calling the createWithDurations function. As a result, no receipt can withdraw funds from SablierV2MerkleLL, and the funds transferred into the contract by the sender will be locked, possibly permanently.

Impact

Missing checks can result in funds being locked up.

Tools Used

manual

Recommendations

It is recommended that when creating the SablierV2MerkleLL contract, check that streamDurations.cliff cannot be greater than streamDurations.total

Updates

Lead Judging Commences

inallhonesty Lead Judge over 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.