Skip to content
This repository has been archived by the owner on Apr 23, 2024. It is now read-only.

Possible DoS for Stellar withdrawals #83

Open
tamirms opened this issue Jul 4, 2022 · 1 comment
Open

Possible DoS for Stellar withdrawals #83

tamirms opened this issue Jul 4, 2022 · 1 comment

Comments

@tamirms
Copy link
Contributor

tamirms commented Jul 4, 2022

I realized that sourceAccount.LastModifiedLedger is updated also when the account receives the deposit not only when the sequence is bumped. With the current single condition I think it's possible to block the withdrawal of any account by sending 1 stroop every ledger. I wonder how and if we can solve this.

EDIT: Looks like we could use CAP-21 and it's seqLedger which equals to "ledger number at which seqNum took on its present value". In Horizon it's Account.SequenceLedger but can be empty if not set in extension.

Originally posted by @bartekn in #68 (comment)

@bartekn
Copy link
Contributor

bartekn commented Jul 4, 2022

Probably solved by 90255c8 in #68.

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

2 participants