diff options
| author | fanquake <fanquake@gmail.com> | 2023-04-11 15:17:41 +0100 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2023-04-11 16:17:04 +0100 |
| commit | c17d4d3b6b5b28a404cf43ce351d5a482d56cc65 (patch) | |
| tree | cb7728ef6fb5c45e414410a8f9af7b6a988ff0fe /src/Makefile.test.include | |
| parent | 53eb4b7a212db7563291509ff4e53327440a9df2 (diff) | |
| parent | 3153e7d779ac284f86e433af033d63f13f361b6f (diff) | |
Merge bitcoin/bitcoin#26662: fuzz: Add HeadersSyncState target
3153e7d779ac284f86e433af033d63f13f361b6f [fuzz] Add HeadersSyncState target (dergoegge)
53552affca381cdb5103ecdbcc7f3fb562e66ac4 [headerssync] Make m_commit_offset protected (dergoegge)
Pull request description:
This adds a fuzz target for the `HeadersSyncState` class.
I am unsure how well this is able to cover the logic since it is just processing unserialized CBlockHeaders straight from the fuzz input (headers are sometimes made continuous). However, it does manage to get to the redownload phase so i thought it is better then not having fuzzing at all.
It would also be nice to fuzz the p2p logic that is using `HeadersSyncState` (e.g. `TryLowWorkHeadersSync`, `IsContinuationOfLowWorkHeadersSync`) but that likely requires some more work (refactoring👻).
ACKs for top commit:
mzumsande:
ACK 3153e7d779ac284f86e433af033d63f13f361b6f
Tree-SHA512: 8a4630ceeeb30e4eeabaa8eb5491d98f0bf900efe7cda07384eaac9f2afaccfbcaa979cc1cc7f0b6ca297a8f5c17a7759f94809dd87eb87d35348d847c83e8ab
Diffstat (limited to 'src/Makefile.test.include')
| -rw-r--r-- | src/Makefile.test.include | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/Makefile.test.include b/src/Makefile.test.include index a39b0abd9d..15d5a17cec 100644 --- a/src/Makefile.test.include +++ b/src/Makefile.test.include @@ -272,6 +272,7 @@ test_fuzz_fuzz_SOURCES = \ test/fuzz/flatfile.cpp \ test/fuzz/float.cpp \ test/fuzz/golomb_rice.cpp \ + test/fuzz/headerssync.cpp \ test/fuzz/hex.cpp \ test/fuzz/http_request.cpp \ test/fuzz/i2p.cpp \ |