Page MenuHomePhabricator

[backport#17083] tests: Add fuzzing harness for various CScript related functions
ClosedPublic

Authored by majcosta on Fri, Jul 10, 18:50.

Details

Summary

dc2fdb99072b87d34620084b82a494a5e698c279 tests: Add fuzzing harness for various CScript related functions (practicalswift)

Pull request description:

Add fuzzing harness for various `CScript` related functions.

**Testing this PR**

Run:

```
$ CC=clang CXX=clang++ ./configure --enable-fuzz --with-sanitizers=address,fuzzer,undefined
$ make
$ src/test/fuzz/script
…
# And to to quickly verify that the relevant code regions are triggered, that the
# fuzzing throughput seems reasonable, etc.
$ contrib/devtools/test_fuzzing_harnesses.sh '^script$'
```

`test_fuzzing_harnesses.sh` can be found in PR #17000.

Backport of Core PR17083

Test Plan
cmake -GNinja .. -DENABLE_SANITIZERS="address;fuzzer" -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++
ninja bitcoin-fuzzers
./src/test/fuzz/script

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

majcosta created this revision.Fri, Jul 10, 18:50
Herald added a reviewer: Restricted Project. · View Herald TranscriptFri, Jul 10, 18:50
majcosta requested review of this revision.Fri, Jul 10, 18:50
teamcity edited the summary of this revision. (Show Details)Fri, Jul 10, 19:10

[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.

deadalnix accepted this revision.Fri, Jul 10, 22:42
This revision is now accepted and ready to land.Fri, Jul 10, 22:42
This revision was landed with ongoing or failed builds.Mon, Jul 13, 19:08
This revision was automatically updated to reflect the committed changes.