Skip to content

Commit db94d74

Browse files
author
MarcoFalke
committed
Merge bitcoin/bitcoin#22657: fuzz: Re-enable assert in banman again
fabed98 fuzz: Re-enable assert in banman again (MarcoFalke) Pull request description: Looks like this was accidentally fixed by removing the buggy code in commit efd6f90 ACKs for top commit: hebasto: ACK fabed98 Tree-SHA512: 2dea5dad48ff2050ae7086c1c6306c40f650e9629918e79adc54164a375d777a70b29f5a480566dc6430f07ce33dfe703fc5d45a20125584b4a026c5832198a2
2 parents 03826ae + fabed98 commit db94d74

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

src/test/fuzz/banman.cpp

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -108,9 +108,7 @@ FUZZ_TARGET_INIT(banman, initialize_banman)
108108
BanMan ban_man_read{banlist_file, /* client_interface */ nullptr, /* default_ban_time */ 0};
109109
banmap_t banmap_read;
110110
ban_man_read.GetBanned(banmap_read);
111-
// Assert temporarily disabled to allow the remainder of the fuzz test to run while a
112-
// fix is being worked on. See https://github.com/bitcoin/bitcoin/pull/22517
113-
(void)(banmap == banmap_read);
111+
assert(banmap == banmap_read);
114112
}
115113
}
116114
fs::remove(banlist_file.string() + ".json");

0 commit comments

Comments
 (0)