fuzz: Misc refactor #21553
Merged
fuzz: Misc refactor #21553
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
maflcko
force-pushed
the
2103-fuzzCleanups
branch
from
fa7b46b to
faa8b2d
Compare
Contributor
|
cr ACK faa8b2de6b9749d33b4f11155a29d7094f23a144: patch looks correct and touches only Thanks for cleaning up! |
maflcko
force-pushed
the
2103-fuzzCleanups
branch
from
faa8b2d to
faaf133
Compare
Contributor
|
cr re-ACK faaf133e4ada4a12783e7395a17f207f7d700e09 |
… target Can be reviewed with --color-moved=dimmed-zebra
maflcko
force-pushed
the
2103-fuzzCleanups
branch
from
faaf133 to
fa4926c
Compare
Member
Author
|
Sorry for the force push. Should be done now. No change in coverage:
|
Contributor
|
cr re-ACK fa4926c |
sidhujag
pushed a commit
to syscoin/syscoin
that referenced
this pull request
Mar 30, 2021
maflcko
pushed a commit
to bitcoin-core/gui
that referenced
this pull request
May 19, 2021
…e CheckTxInputs fae4ee5 fuzz: Add missing CheckTransaction before CheckTxInputs (MarcoFalke) faacb7e fuzz: Sanity check result of CheckTransaction (MarcoFalke) Pull request description: This bug was introduced by myself in commit eeee8f5 (bitcoin/bitcoin#21553) Reproducer: https://github.com/bitcoin/bitcoin/files/6492249/clusterfuzz-testcase-minimized-coins_view-6109460079706112.log Hopefully fixes https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=34301 ACKs for top commit: practicalswift: cr ACK fae4ee5: patch looks correct :) Tree-SHA512: 9ece7a5c4bfa60f5e5ffeba3f0ee52a07944c9bd6102588dd7ff7405695e6b32449945b7c41bd25baf38814df5a2436521e655ceff87223ad03c69ed39053023
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Some small refactors to remove unused and redundant fuzz code