sikkienl/dash
1
0
Fork 0
mirror of https://github.com/dashpay/dash.git synced 2024-12-26 04:22:55 +01:00
Code Issues Actions Packages Projects Releases Wiki Activity

Merge bitcoin/bitcoin#22657: fuzz: Re-enable assert in banman again

Browse Source 
fabed982ad9143cddaca8346f6b4c243dd84e0c4 fuzz: Re-enable assert in banman again (MarcoFalke)

Pull request description:

  Looks like this was accidentally fixed by removing the buggy code in commit efd6f904c78769ad2e93c1f1de43014d284e7561

ACKs for top commit:
  hebasto:
    ACK fabed982ad9143cddaca8346f6b4c243dd84e0c4

Tree-SHA512: 2dea5dad48ff2050ae7086c1c6306c40f650e9629918e79adc54164a375d777a70b29f5a480566dc6430f07ce33dfe703fc5d45a20125584b4a026c5832198a2
This commit is contained in:
MarcoFalke 2021-08-07 16:28:45 +02:00 committed by pasta
parent 83e374f081
commit e40d67a170
No known key found for this signature in database
GPG Key ID: 52527BEDABE87984
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/test/fuzz/banman.cpp
View File

@ -107,9 +107,7 @@ 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 temporarily disabled to allow the remainder of the fuzz test to run while a
// fix is being worked on. See https://github.com/bitcoin/bitcoin/pull/22517
(void)(banmap == banmap_read);
assert(banmap == banmap_read);
}
}
fs::remove(banlist_file.string() + ".json");