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.
fuzz: remove GetDescriptorChecksum from string harness #34985
pull brunoerg wants to merge 1 commits into bitcoin:master from brunoerg:2026-04-fuzz-remove-descriptor-from-string changing 1 files +0 −1-
brunoerg commented at 1:44 PM on April 1, 2026: contributor
-
91cd0e3aaa
fuzz: remove GetDescriptorChecksum from string harness
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.
- DrahtBot added the label Fuzzing on Apr 1, 2026
-
DrahtBot commented at 1:44 PM on April 1, 2026: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process.
Type Reviewers ACK frankomosh, sipa If your review is incorrectly listed, please copy-paste <code><!--meta-tag:bot-skip--></code> into the comment that the bot should ignore.
<!--5faf32d7da4f0f540f40219e4f7537a3-->
-
frankomosh commented at 7:51 PM on April 1, 2026: contributor
utACK 91cd0e3aaa4e69b3571272b1688cba9bead1a294
-
majdalwsoof1232580-hub commented at 9:09 PM on April 5, 2026: none
-
sipa commented at 10:15 PM on April 5, 2026: member
ACK 91cd0e3aaa4e69b3571272b1688cba9bead1a294. I don't think this tests anything useful.
- ryanofsky assigned ryanofsky on Apr 7, 2026
- ryanofsky merged this on Apr 7, 2026
- ryanofsky closed this on Apr 7, 2026
- brunoerg deleted the branch on Apr 7, 2026
Labels