-
Notifications
You must be signed in to change notification settings - Fork 37.7k
tests: Add fuzzing harness for various CScript related functions #17083
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers. ConflictsReviewers, this pull request conflicts with the following ones:
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. |
b547b53
to
edc77a4
Compare
edc77a4
to
b6de958
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK b6de958
Show signature and timestamp
Signature:
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512
ACK b6de95843982f7bf450fd03f8d71577a6f24d1db
-----BEGIN PGP SIGNATURE-----
iQGzBAEBCgAdFiEE+rVPoUahrI9sLGYTzit1aX5ppUgFAlwqrYAACgkQzit1aX5p
pUgkvgwAiGdaNmf4ABeRBuPLcEFRJ285f2hJg8Wf8Gcmwop6ExPT0Sue4eN+VRYc
t9Zlm6ZEZqyK6chPafijQ9nrbHsFUEbibCUT9hZ7FJgSoWOxh7ahyPaN00HXjucV
bFBVLUfdJ/N3F3in2XFFiWuK2KAHkRCnf642FmeZ0uM0QVvjOdYJzB51JZaXbRrJ
h8sRqGL6+Wd/m2lADT0454rkUYpkrvqfN7K2Y9M7yY8Ey7VmFR9Jpp8mcQeEWiEp
BmZ4MwBg9xpzX2DnxDuT/ajja8kzgv2VeY/M3jbAB/BYpeikuutlUUeoydTs/aZ7
8L6oOsxOP99vL2Aq1CWOb8m8g5S2IMmS6c0xWXwvrjWEXPDNo+FlfaYbBI6P9wJS
9w8xfzxQd+/CXB3nLAr5ImNnOz2Qft6kCeqxOWGDWV7Rd87VT/UYjOJkuPienzsl
Y4fq9mvlGjaUJbtSf3JHD+S5e2DnCTxAkiqzXqrGW8gMkw4uY+KxcUMhPvjqvfCL
Pou3ZL9B
=sD4t
-----END PGP SIGNATURE-----
Timestamp of file with hash 7b53754fa9e50b769d383a78f34b54a3e8ff4ddd3b902ddcb99121307e7f81e6 -
ddf080c
to
fe2118f
Compare
Rebased and added |
fe2118f
to
dc2fdb9
Compare
…functions dc2fdb9 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. Top commit has no ACKs. Tree-SHA512: a0c5dca3b64ae177020b2ca299a29015d70755231b6bf01edbfc67c8aac90c44b1b4d57350c3aebef6e031108e6ae8e5fa0987c67707831c314f5d3090e0cee8
…ed functions 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 [[bitcoin/bitcoin#17083 | PR17083]] Test Plan: cmake -GNinja .. -DENABLE_SANITIZERS="address;fuzzer" -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++ ninja bitcoin-fuzzers ./src/test/fuzz/script Reviewers: #bitcoin_abc, deadalnix Reviewed By: #bitcoin_abc, deadalnix Differential Revision: https://reviews.bitcoinabc.org/D6896
…elated functions dc2fdb9 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 bitcoin#17000. Top commit has no ACKs. Tree-SHA512: a0c5dca3b64ae177020b2ca299a29015d70755231b6bf01edbfc67c8aac90c44b1b4d57350c3aebef6e031108e6ae8e5fa0987c67707831c314f5d3090e0cee8
Add fuzzing harness for various
CScript
related functions.Testing this PR
Run:
test_fuzzing_harnesses.sh
can be found in PR #17000.