-
Notifications
You must be signed in to change notification settings - Fork 38.6k
tests: Add proof-of-work fuzzing harness #18407
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. ConflictsNo conflicts as of last run. |
|
Concept ACK, commenting to be notified when this is rebased. |
6ca8d0a to
baa778a
Compare
|
@jonasnick Rebased! :) |
jonatack
left a comment
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 baa778a
bitcoin/bitcoin (pr/18407)$ src/test/fuzz/pow
INFO: Seed: 3362987756
INFO: Loaded 1 modules (514213 inline 8-bit counters): 514213 [0x562fb3600568, 0x562fb367de0d),
INFO: Loaded 1 PC tables (514213 PCs): 514213 [0x562fb367de10,0x562fb3e56860),
INFO: -max_len is not provided; libFuzzer will not generate inputs larger than 4096 bytes
INFO: A corpus is not provided, starting from an empty corpus
#2 INITED cov: 140 ft: 141 corp: 1/1b exec/s: 0 rss: 116Mb
#5 NEW cov: 140 ft: 148 corp: 2/3b exec/s: 0 rss: 117Mb L: 2/2 MS: 3 ChangeByte-ShuffleBytes-CopyPart-
#8 NEW cov: 140 ft: 153 corp: 3/6b exec/s: 0 rss: 117Mb L: 3/3 MS: 3 EraseBytes-InsertByte-InsertByte-
NEW_FUNC[0/35]: 0x562fb06f9180 in boost::optional<CBlockHeader> ConsumeDeserializable<CBlockHeader>(FuzzedDataProvider&, unsigned long) /home/jon/projects/bitcoin/bitcoin/src/./test/fuzz/util.h:42
NEW_FUNC[1/35]: 0x562fb06fcf30 in FuzzedDataProvider::Advance(unsigned long) /home/jon/projects/bitcoin/bitcoin/src/./test/fuzz/FuzzedDataProvider.h:247
#14 NEW cov: 460 ft: 512 corp: 4/81b exec/s: 0 rss: 118Mb L: 75/75 MS: 1 InsertRepeatedBytes-
.../...
#220985 REDUCE cov: 1308 ft: 7563 corp: 309/508Kb exec/s: 373 rss: 517Mb L: 869/4096 MS: 3 ChangeASCIIInt-ChangeBit-EraseBytes-
#221115 REDUCE cov: 1308 ft: 7563 corp: 309/508Kb exec/s: 374 rss: 517Mb L: 147/4096 MS: 5 PersAutoDict-ChangeByte-CopyPart-EraseBytes-CopyPart- DE: "\x08\x00\x00\x00"-
|
@MarcoFalke Would you mind reviewing? :) |
maflcko
left a comment
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 baa778ae1a7d75a448bafe30115f963e7de1fc18
baa778a to
acf269e
Compare
|
@MarcoFalke Updated. Please re-review :) |
Summary: Backport of core [[bitcoin/bitcoin#18407 | PR18407]]. Depends on D8981. Test Plan: ninja bitcoin-fuzzers ./test/test/fuzz/pow <path_to_corpus> Reviewers: #bitcoin_abc, PiRK Reviewed By: #bitcoin_abc, PiRK Differential Revision: https://reviews.bitcoinabc.org/D8984
backport: bitcoin#18417, bitcoin#18521, bitcoin#18529, bitcoin#18176, bitcoin#18423, bitcoin#17926, bitcoin#18353, bitcoin#18407, bitcoin#18455, bitcoin#18565, bitcoin#18867 (fuzzing harness backports: part 2)
Add proof-of-work fuzzing harness.