Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Vesting merge_schedules should be added to the NonTransfer proxy filter #3436

Open
emostov opened this issue Jul 8, 2021 · 0 comments
Open
Assignees

Comments

@emostov
Copy link
Contributor

emostov commented Jul 8, 2021

paritytech/substrate#9202 + #3407 will add the ability to merge two vesting schedules. This call should be added to the NonTransfer proxy filter.

The main attack with merge schedules I can think of is to send an account a schedule that starts and ends at BlockNumber::max_value and then merge all there schedules with that one, which will make sure they can't unlock funds until block height hits BlockNumber::max_value.

After some quick discussion with @shawntabrizi I think this risk profile is in line with some other permitted calls for NonTransfer.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant