Skip to content

Commit 1231278

Browse files
MarcoFalkePastaPastaPasta
authored andcommitted
Merge bitcoin#22517: fuzz: Temporarily disable failing assert in banman fuzz test
fa8bed6 fuzz: Temporarily disable failing assert in banman fuzz test (MarcoFalke) Pull request description: Otherwise the remainder of the fuzz test can't be fuzzed without running into crashes ACKs for top commit: practicalswift: cr ACK fa8bed6 Tree-SHA512: ec6606292e2cfd26484c7f6caf1c418c377da54111b332990fce68373f0438defda71d931a42ca34431527fbc172dd2fdf29b260afca15b34910ee137de1c365
1 parent 6ba92a6 commit 1231278

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

src/test/fuzz/banman.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,8 @@ FUZZ_TARGET_INIT(banman, initialize_banman)
107107
BanMan ban_man_read{banlist_file, /* client_interface */ nullptr, /* default_ban_time */ 0};
108108
banmap_t banmap_read;
109109
ban_man_read.GetBanned(banmap_read);
110-
assert(banmap == banmap_read);
110+
// Temporarily disabled to allow the remainder of the fuzz test to run while a fix is being worked on:
111+
// assert(banmap == banmap_read);
111112
}
112113
}
113114
fs::remove(banlist_file.string() + ".json");

0 commit comments

Comments
 (0)