Sablier

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

missing important check on multiple _create functions leading to unintended behaviour

Summary

missing check that funder is not the recipient in _create()

Vulnerability Details

a user can make mistakes while setting any of the params, thats why we do multiple checks in different places like here and here
but the function doesn't ever check if the recipient != msg.sender which may happen by mistake cause no protocol owner ever want to airdrop his wallet from his deducted balance

Impact

unintended behavior and forced waiting lock period to retrieve funds

Tools Used

manual review

Recommendations

add a requirement that funder != receiver

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.