Should we remove the deltaCount
check?
#96
Closed
andreivladbrg
started this conversation in
General
Replies: 2 comments
-
This is an interesting idea - I have to sleep on it though. You're right that the call would revert anyway. But I'm trying to think if there's any way this would open up an exploit .. Lemme think. |
Beta Was this translation helpful? Give feedback.
0 replies
-
Created an issue for this, locking. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Given that the
createInternal
function callscheckSegmentCounts
function that checks if the counts of segments match and if the counts are less than theMAX_SEGMENT_COUNT
, I don't think this check from thecreateWithDuration
function is still necessary:https://github.com/sablierhq/v2-core/blob/5f13c560f6a3c99a10899ffc95907a8bc7414d84/src/SablierV2Pro.sol#L238-L241
Beta Was this translation helpful? Give feedback.
All reactions