This should help to get fuzz coverage for https://maflcko.github.io/b-c-cov/fuzz.coverage/src/streams.cpp.gcov.html
Also, remove unused code and fix a timeout bug.
fuzz: AutoFile with XOR #28873
pull maflcko wants to merge 3 commits into bitcoin:master from maflcko:2311-fuzz-file- changing 7 files +29 −43-
maflcko commented at 4:22 PM on November 14, 2023: member
-
DrahtBot commented at 4:22 PM on November 14, 2023: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--006a51241073e994b41acfe9ec718e94-->
Code Coverage
For detailed information about the code coverage, see the test coverage report.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process.
Type Reviewers ACK dergoegge If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
- DrahtBot renamed this:
fuzz: AutoFile with XOR
fuzz: AutoFile with XOR
on Nov 14, 2023 - DrahtBot added the label Tests on Nov 14, 2023
-
fa5388fad3
fuzz: Remove FuzzedAutoFileProvider
The code is clearer without it. This is also needed for a future commit.
-
fab5cb9066
fuzz: Reduce LIMITED_WHILE limit for file fuzzing
A higher limit is not needed, and only leads to timeouts, see for example the buffered_file one in https://github.com/bitcoin/bitcoin/issues/28812#issue-1981386486
-
fuzz: AutoFile with XOR faa25718b3
- maflcko force-pushed on Nov 14, 2023
-
dergoegge commented at 5:02 PM on November 14, 2023: member
Concept ACK
- dergoegge approved
-
dergoegge commented at 4:14 PM on November 16, 2023: member
ACK faa25718b3f11f24aa41f0968bbd4da104814bc5
Individual coverage after brief fuzzing:
- fanquake merged this on Nov 17, 2023
- fanquake closed this on Nov 17, 2023
- maflcko deleted the branch on Nov 17, 2023
- bitcoin locked this on Nov 16, 2024
