Skip to content

Commit 218fe60

Browse files
author
MarcoFalke
committed
Merge #20290: fuzz: Fix DecodeHexTx fuzzing harness issue
28f8cb1 fuzz: Fix DecodeHexTx fuzzing harness issue (practicalswift) Pull request description: Fix `DecodeHexTx` fuzzing harness issue. Before this patch: ``` $ src/test/fuzz/decode_tx decode_tx: test/fuzz/decode_tx.cpp:29: void test_one_input(const std::vector<uint8_t> &): Assertion `result_try_witness_and_maybe_no_witness' failed. … ``` After this patch: ``` $ src/test/fuzz/decode_tx … ``` ACKs for top commit: MarcoFalke: review ACK 28f8cb1 Tree-SHA512: 2ed11b2f00a4c6fa3e8eea76a2a37d89a4b8d52815264676fe3de0a26ad7906cfafda9b843ceede2fd428815472e01fd1f87afb851282a8c7839bd4c87dc382b
2 parents 5174b53 + 28f8cb1 commit 218fe60

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

src/test/fuzz/decode_tx.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,7 @@ void test_one_input(const std::vector<uint8_t>& buffer)
2424
if (result_try_witness_and_maybe_no_witness) {
2525
assert(result_try_no_witness || result_try_witness);
2626
}
27-
// if (result_try_no_witness) { // Uncomment when https://github.com/bitcoin/bitcoin/pull/17775 is merged
28-
if (result_try_witness) { // Remove stop-gap when https://github.com/bitcoin/bitcoin/pull/17775 is merged
27+
if (result_try_no_witness) {
2928
assert(result_try_witness_and_maybe_no_witness);
3029
}
3130
}

0 commit comments

Comments
 (0)