This is not an actual fuzz target. It is a hack to exploit the built-in capability of fuzz engines to measure coverage.
fuzz: Hide script_assets_test_minimizer #20663
pull MarcoFalke wants to merge 2 commits into bitcoin:master from MarcoFalke:2012-fuzzNoReg changing 3 files +25 −20-
MarcoFalke commented at 6:20 PM on December 15, 2020: member
-
MarcoFalke commented at 6:20 PM on December 15, 2020: member
-
MarcoFalke commented at 6:22 PM on December 15, 2020: member
Can be tested by running
PRINT_ALL_FUZZ_TARGETS_AND_ABORT=1 ./src/test/fuzz/fuzz | grep min | wcand observing zeros -
sipa commented at 6:25 PM on December 15, 2020: member
Wouldn't it be better to leave it as-is, and just exclude it from being listed, or exclude it test-runner.py? That way you don't need to modify the source code in order to use it.
- DrahtBot added the label Tests on Dec 15, 2020
- MarcoFalke renamed this:
fuzz: Leave script_assets_test_minimizer unregistered
fuzz: Hide script_assets_test_minimizer
on Dec 16, 2020 - MarcoFalke force-pushed on Dec 16, 2020
- MarcoFalke force-pushed on Dec 16, 2020
-
practicalswift commented at 1:19 PM on December 16, 2020: contributor
Concept ACK: thanks for addressing this!
- DrahtBot cross-referenced this on Dec 21, 2020 from issue inline non-member functions with body in fuzzing headers by pstratem
-
DrahtBot commented at 7:48 AM on December 21, 2020: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--174a7506f384e20aa4161008e828411d-->
Conflicts
No conflicts as of last run.
- MarcoFalke force-pushed on Dec 21, 2020
- MarcoFalke cross-referenced this on Dec 23, 2020 from issue Add merged seeds. by pstratem
- DrahtBot cross-referenced this on Jan 3, 2021 from issue fuzz: Avoid extraneous copy of input data, using Span<> by MarcoFalke
- DrahtBot added the label Needs rebase on Feb 5, 2021
- MarcoFalke force-pushed on Feb 8, 2021
-
fafca47adc
fuzz: Hide script_assets_test_minimizer
Can be reviewed with --ignore-all-space
-
doc: Fixup docs in fuzz/script_assets_test_minimizer.cpp fac726b1b8
- MarcoFalke force-pushed on Feb 8, 2021
- MarcoFalke cross-referenced this on Feb 8, 2021 from issue unit test data: add new taproot assets by MarcoFalke
- DrahtBot removed the label Needs rebase on Feb 8, 2021
-
practicalswift commented at 9:08 PM on February 10, 2021: contributor
cr ACK fac726b1b8331b267973138bbd2bff5304774315: patch looks correct and touches only
src/test/fuzz/ - MarcoFalke merged this on Feb 11, 2021
- MarcoFalke closed this on Feb 11, 2021
- MarcoFalke deleted the branch on Feb 11, 2021
- sidhujag referenced this in commit 74d4cc59d4 on Feb 11, 2021
- bitcoin locked this on Aug 16, 2022
Contributors
Labels