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

fuzz: Fix tx_pool target to properly fuzz immature outpoints #21512

Merged
merged 2 commits into from Mar 23, 2021

Conversation

maflcko
Copy link
Member

@maflcko maflcko commented Mar 23, 2021

Also includes a commit for minor style fixups

@fanquake fanquake added the Tests label Mar 23, 2021
@maflcko
Copy link
Member Author

maflcko commented Mar 23, 2021

Bug was found by @glozow in #21142 (comment) Thanks for the review!

@laanwj laanwj changed the title fuzz: Fix tx_pool target to properly fuzz immature outpoints fuzz: Fix tx_pool target to properly fuzz immature outpoints Mar 23, 2021
Copy link
Member

@glozow glozow left a comment

Choose a reason for hiding this comment

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

utACK fac921f this fixes it 👍

@maflcko maflcko merged commit 681c21b into bitcoin:master Mar 23, 2021
@maflcko maflcko deleted the 2103-fuzzImmaturePool branch March 23, 2021 15:06
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Mar 23, 2021
…re outpoints

fac921f fuzz: Fix tx_pool target to properly fuzz immature outpoints (MarcoFalke)
fa2b95f fuzz: Style fixups (MarcoFalke)

Pull request description:

  Also includes a commit for minor style fixups

ACKs for top commit:
  glozow:
    utACK fac921f this fixes it 👍

Tree-SHA512: 1575ba115b2009b653921511c163bd846cd381d6fc92b04a899c0686d23a02bdcdd95c81776b515b80ae187bcec3ccaca3aa88fcecbec888f73ca2d875eef506
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Aug 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants