Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix crash when creating large inputs #1014

Merged

Conversation

apoelstra
Copy link
Member

There isn't really any reasonable way to recover from this situation but we can at least gracefully fail instead of triggering an assertation failure.

Fixes #979

Fixes #880

@apoelstra apoelstra force-pushed the 2021-06--fix-surjectionproof-crash branch from f806047 to f7e1fb6 Compare July 1, 2021 22:04
@apoelstra
Copy link
Member Author

Rebased

@gwillen
Copy link
Contributor

gwillen commented Aug 30, 2021

utACK f7e1fb6.

Thanks, this was really annoying.

@apoelstra apoelstra merged commit d973e11 into ElementsProject:master Aug 30, 2021
apoelstra added a commit to apoelstra/elements that referenced this pull request Aug 31, 2021
apoelstra added a commit that referenced this pull request Aug 31, 2021
…ew RPC changes

c0936ba test: fix surjectionproof overflow functional test for new RPC changes (Andrew Poelstra)

Pull request description:

  This fixes an undetected merge conflict between #1014 and #900.

ACKs for top commit:
  gwillen:
    utACK c0936ba
  psgreco:
    UTACK c0936ba

Tree-SHA512: a39d98db91d92f7e0e739e06d7a1bb13814d2bc7751cb6105a759af04ed0d61d206d1650f5e4e087feaeafcf1c482587b05b66d9cc497bc594a45df6bf23c07a
gwillen pushed a commit that referenced this pull request Jun 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Creating a transaction with too many inputs crashes QT abort when blinding very large transaction
2 participants