Skip to content

Call existsProviderTxConflict after CheckSpecialTx#2690

Merged
codablock merged 1 commit into
dashpay:developfrom
codablock:pr_fix_mempool_crash
Feb 6, 2019
Merged

Call existsProviderTxConflict after CheckSpecialTx#2690
codablock merged 1 commit into
dashpay:developfrom
codablock:pr_fix_mempool_crash

Conversation

@codablock

Copy link
Copy Markdown

Otherwise we might end up passing and invalid proTx into it, causing
assertions to fail and thus crash the process.

This should also be backported to v13.

Otherwise we might end up passing and invalid proTx into it, causing
assertions to fail and thus crash the process.

@UdjinM6 UdjinM6 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

@codablock
codablock merged commit 5478183 into dashpay:develop Feb 6, 2019
@codablock
codablock deleted the pr_fix_mempool_crash branch February 6, 2019 16:54
codablock added a commit to codablock/dash that referenced this pull request Feb 6, 2019
Otherwise we might end up passing and invalid proTx into it, causing
assertions to fail and thus crash the process.
CryptoCentric pushed a commit to absolute-community/absolute that referenced this pull request Jun 29, 2020
Otherwise we might end up passing and invalid proTx into it, causing
assertions to fail and thus crash the process.
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.

2 participants