Differential D9111

tests: Add fuzzing harnesses for functions in script/
ClosedPublic
Actions

Authored by Fabien on Jan 29 2021, 09:52.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABCe9af331a7784: tests: Add fuzzing harnesses for functions in script/

Summary

- Add fuzzing helper functions ConsumeDataStream and ConsumeUInt160
- Fill fuzzing coverage gaps for functions in script/script.h,
  script/script_error.h and script/standard.h
- Add fuzzing harness for functions in script/bitcoinconsensus.h
- Add fuzzing harness for functions in script/descriptor.h
- Add fuzzing harness for functions in script/interpreter.h
- Add fuzzing harness for functions in script/sigcache.h
- Add fuzzing harness for functions in script/sign.h

Backport of core PR18994.

Test Plan

ninja bitcoin-fuzzers
./test/fuzz/test_runner.py <path_to_corpus>

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Jan 29 2021, 09:52

Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 29 2021, 09:52

Fabien requested review of this revision.Jan 29 2021, 09:52

teamcity edited the summary of this revision. (Show Details)Jan 29 2021, 09:52

Harbormaster completed remote builds in B15052: Diff 27425.Jan 29 2021, 10:07

PiRK accepted this revision.Jan 29 2021, 14:31

This revision is now accepted and ready to land.Jan 29 2021, 14:31

Closed by commit rABCe9af331a7784: tests: Add fuzzing harnesses for functions in script/ (authored by practicalswift <practicalswift@users.noreply.github.com>, committed by Fabien). · Explain WhyJan 29 2021, 19:12

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABCe9af331a7784: tests: Add fuzzing harnesses for functions in script/.

Revision Contents
Changeset List

Path

Size

src/

test/

fuzz/

CMakeLists.txt

10 lines

script.cpp

28 lines

script_bitcoin_consensus.cpp

44 lines

script_descriptor_cache.cpp

49 lines

script_interpreter.cpp

59 lines

52 lines

185 lines

5 lines

56 lines

Diff 27433

View Options

src/test/fuzz/CMakeLists.txt

View Options

src/test/fuzz/script.cpp

View Options

src/test/fuzz/script_bitcoin_consensus.cpp

View Options

src/test/fuzz/script_descriptor_cache.cpp

View Options

src/test/fuzz/script_interpreter.cpp

View Options

src/test/fuzz/script_sigcache.cpp

View Options

src/test/fuzz/script_sign.cpp

View Options

src/test/fuzz/string.cpp

View Options

tests: Add fuzzing harnesses for functions in script/ClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 27433

src/test/fuzz/CMakeLists.txt

src/test/fuzz/script.cpp

src/test/fuzz/script_bitcoin_consensus.cpp

src/test/fuzz/script_descriptor_cache.cpp

src/test/fuzz/script_interpreter.cpp

src/test/fuzz/script_sigcache.cpp

src/test/fuzz/script_sign.cpp

src/test/fuzz/string.cpp

src/test/fuzz/util.h

tests: Add fuzzing harnesses for functions in script/
ClosedPublic
Actions

Revision Contents
Changeset List