Skip to content

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