tests: Add fuzzing harness for descriptor Span-parsing helpers#17113
Merged
maflcko merged 1 commit intobitcoin:masterfrom Oct 16, 2019
Merged
tests: Add fuzzing harness for descriptor Span-parsing helpers#17113maflcko merged 1 commit intobitcoin:masterfrom
maflcko merged 1 commit intobitcoin:masterfrom
Conversation
Contributor
|
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. |
maflcko
reviewed
Oct 15, 2019
maflcko
reviewed
Oct 15, 2019
Member
maflcko
left a comment
maflcko
left a comment
There was a problem hiding this comment.
ACK 0e2585875764d1c120043f0e9a3a499b0079726b only, did not look at the other commits
Show signature and timestamp
Signature:
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512
ACK 0e2585875764d1c120043f0e9a3a499b0079726b only, did not look at the other commits
-----BEGIN PGP SIGNATURE-----
iQGzBAEBCgAdFiEE+rVPoUahrI9sLGYTzit1aX5ppUgFAlwqrYAACgkQzit1aX5p
pUjAeAwAr9NgrYNFJbdZmonusrCxHskWSxI5FNPfveSJeSEoltttDv46rvXU2MOF
FCQ0WS8vaUHV/9t/5R8AmTGvB91I5q/YkVd/6dCL3TTupiDQS25ofHy5a09D+2xi
XE69KSQ7hNkS9IvEjaO5xhcYumh+Q5ZZQQjDgwvKvNbX72F5YBXZc6sAg4Iz+9/L
ke/7bVIshN3ixSe0qEPWctx6Kmp64sBq0CPGLGiH368wXtQcy07bpzMAndJXjBhD
rjl/kM7oPzp+JXVa64+rOIHpDKWHx06ckTR1z/7+Ju2St8SIHzQq1NGjYcPhR5RO
/C/4NQBVeHklYZSyGjjSddEcCR2a+rirzIOTIUab5jqKPH7XMEGBaPUVMyhS5j9g
lyTPMYzJOreTi9oW/WwwA2DM5xWqVJO9+jtEXZft6r9Yecux0+/uElAzzytn1YwN
7Ty/HTNJNQ7pgKrGauxZEQ3uPZkMgVKJhYxzCcE3HWYV7P8rQd+s/aGmM5Xw3Dk0
r3odoobi
=/nL5
-----END PGP SIGNATURE-----
Timestamp of file with hash 31bdd75e9fd947f3bb361daae0a144d145e489f6aad21993c7234511eac4cd3c -
practicalswift
force-pushed
the
fuzzers-spanparsing
branch
from
0e25858 to
58d67f1
Compare
Member
|
re-ACK 58d67f1 Show signature and timestampSignature: Timestamp of file with hash |
maflcko
pushed a commit
that referenced
this pull request
Oct 16, 2019
…helpers 58d67f1 tests: Add fuzzing harness for descriptor Span-parsing helpers (practicalswift) Pull request description: Add fuzzing harness for descriptor Span-parsing helpers (`spanparsing`). As suggested by a fuzz testing enthusiast in #16887 (comment). **Testing this PR** Run: ``` $ CC=clang CXX=clang++ ./configure --enable-fuzz \ --with-sanitizers=address,fuzzer,undefined $ make $ src/test/fuzz/spanparsing ``` ACKs for top commit: MarcoFalke: re-ACK 58d67f1 Tree-SHA512: 5eaca9fcda2856e0dcfeb4a98a2dc97051ae6251f7642b92fdae3ff96bb95ccb0377ee4e6c6b531e59061983b8d9485a5282467f2ab1d614861f60202a893b1c
jasonbcox
pushed a commit
to Bitcoin-ABC/bitcoin-abc
that referenced
this pull request
Jul 9, 2020
…ng helpers Summary: 58d67f1cc068c3779e309dc8a82ce33158c3e5b2 tests: Add fuzzing harness for descriptor Span-parsing helpers (practicalswift) Pull request description: Add fuzzing harness for descriptor Span-parsing helpers (`spanparsing`). As suggested by a fuzz testing enthusiast in bitcoin/bitcoin#16887 (comment). **Testing this PR** Run: ``` $ CC=clang CXX=clang++ ./configure --enable-fuzz \ --with-sanitizers=address,fuzzer,undefined $ make $ src/test/fuzz/spanparsing ``` --- Depends on D6879 Backport of Core [[bitcoin/bitcoin#17113 | PR17113]] Test Plan: cmake -GNinja .. -DENABLE_SANITIZERS="address;fuzzer" -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++ ninja bitcoin-fuzzers link-fuzz-test_runner.py run `./src/test/fuzz/spanparsing` for a few seconds Reviewers: #bitcoin_abc, deadalnix Reviewed By: #bitcoin_abc, deadalnix Differential Revision: https://reviews.bitcoinabc.org/D6880
kwvg
added a commit
to kwvg/dash
that referenced
this pull request
Mar 24, 2022
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add fuzzing harness for descriptor Span-parsing helpers (
spanparsing).As suggested by a fuzz testing enthusiast in #16887 (comment).
Testing this PR
Run: