Skip to content

Commit

Permalink
Merge bitcoin#22517: fuzz: Temporarily disable failing assert in banm…
Browse files Browse the repository at this point in the history
…an 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
  • Loading branch information
MarcoFalke authored and vijaydasmp committed Nov 8, 2023
1 parent 409d6a4 commit da9a26f
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/test/fuzz/banman.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,8 @@ FUZZ_TARGET_INIT(banman, initialize_banman)
BanMan ban_man_read{banlist_file, /* client_interface */ nullptr, /* default_ban_time */ 0};
banmap_t banmap_read;
ban_man_read.GetBanned(banmap_read);
assert(banmap == banmap_read);
// Temporarily disabled to allow the remainder of the fuzz test to run while a fix is being worked on:
// assert(banmap == banmap_read);
}
}
fs::remove(banlist_file.string() + ".json");
Expand Down

0 comments on commit da9a26f

Please sign in to comment.