highperfocused/bitcoin
1
0
Fork 0
mirror of https://github.com/bitcoin/bitcoin.git synced 2026-04-09 15:18:03 +02:00
Code Issues Packages Projects Releases Wiki Activity

Merge bitcoin/bitcoin#34985: fuzz: remove GetDescriptorChecksum from string harness

Browse Source 
91cd0e3aaa fuzz: remove GetDescriptorChecksum from string harness (Bruno Garcia)

Pull request description:

  This function is already strongly fuzzed by other harness. E.g: descriptor_parse calls it several times during parsing and serialization. Also, calling GetDescriptorChecksum with a string of length 32 is not effective to exercise it.

ACKs for top commit:
  frankomosh:
    utACK 91cd0e3aaa
  sipa:
    ACK 91cd0e3aaa. I don't think this tests anything useful.

Tree-SHA512: ac5e5c3f04669d4f7c24c25390565b02350ac2d4b1044aa1b3b9b83b3a70e57b3a109bd8d58298e722ce97d4a76efc62f11b31147a810a65db7ac592606acc3c
This commit is contained in:
Ryan Ofsky
2026-04-07 13:33:56 -04:00
parent 485ebad1ee 91cd0e3aaa
commit d868667fdc
1 changed files with 0 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
src/test/fuzz/string.cpp
View File

@@ -64,7 +64,6 @@ FUZZ_TARGET(string)
const auto width{fuzzed_data_provider.ConsumeIntegralInRange<size_t>(1, 1000)};
(void)FormatParagraph(random_string_1, width, fuzzed_data_provider.ConsumeIntegralInRange<size_t>(0, width));
(void)FormatSubVersion(random_string_1, fuzzed_data_provider.ConsumeIntegral<int>(), random_string_vector);
(void)GetDescriptorChecksum(random_string_1);
(void)HelpExampleCli(random_string_1, random_string_2);
(void)HelpExampleRpc(random_string_1, random_string_2);
(void)HelpMessageGroup(random_string_1);