Flow

Sablier
FoundryDeFi
20,000 USDC
View results
Submission Details
Severity: high
Invalid

lack of update of user balance

Summary

Vulnerability Details

https://github.com/Cyfrin/2024-10-sablier/blob/main/src/SablierFlow.sol#L254

this function do not account for users making the same transaction multiple times

this deposite function allow user to deposite in a stream, which users can deposit more than once in a stream but this function do not track user balance, if deposit is called twice the first balance will be overwritten by the second transaction

Impact

if a user should deposit twice the first amount will be overwritten by the second transaction which can lead to lost of fund

Tools Used

manaul

Recommendations

user deposit should be updated

Updates

Lead Judging Commences

inallhonesty Lead Judge
8 months ago
inallhonesty Lead Judge 8 months ago
Submission Judgement Published
Invalidated
Reason: Lack of quality
inallhonesty Lead Judge 8 months ago
Submission Judgement Published
Invalidated
Reason: Lack of quality

Support

FAQs

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