fuzz: Limit fuzz buffer size in script_flags target #28931

pull maflcko wants to merge 3 commits into bitcoin:master from maflcko:2311-fuzz-script-flags- changing 2 files +18 −19

maflcko commented at 4:51 PM on November 23, 2023: member

Most fuzz targets have an upper limit on the buffer size to avoid excessive runtime. Do the same for script_flags to avoid timeouts such as #28812 (comment)

Also, fix iwyu. Also, remove legacy CDataStream.
fuzz: CDataStream -> DataStream in script_flags fa6b87b9ee
DrahtBot commented at 4:51 PM on November 23, 2023: contributor


The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.



Code Coverage

For detailed information about the code coverage, see the test coverage report.



Reviews

See the guideline for information on the review process.

Type Reviewers

ACK dergoegge, brunoerg

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.



Conflicts

Reviewers, this pull request conflicts with the following ones:
- #28451 (refactor: Remove unused SER_DISK, SER_NETWORK, SER_GETHASH by maflcko)
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
DrahtBot added the label Tests on Nov 23, 2023
fuzz: Limit fuzz buffer size in script_flags target fa71285b73

Type	Reviewers
ACK	dergoegge, brunoerg

Fix IWYU for the script_flags fuzz target

Also, export script_error.h from interpreter.h, because there should
rarely be a case where script_error.h is included without interpreter.h

faf1fb207f

maflcko force-pushed on Nov 23, 2023
fanquake requested review from dergoegge on Nov 23, 2023
dergoegge approved
dergoegge commented at 10:25 AM on November 24, 2023: member

ACK faf1fb207fb6e9a12c864074f8c40d5922d93ff4
brunoerg approved
brunoerg commented at 10:11 PM on November 25, 2023: contributor

utACK faf1fb207fb6e9a12c864074f8c40d5922d93ff4
fanquake merged this on Nov 26, 2023
fanquake closed this on Nov 26, 2023
maflcko deleted the branch on Nov 27, 2023
bitcoin locked this on Nov 26, 2024

Contributors

Labels