Differential D8979

tests: Add fuzzing harnesses for functions/classes in chain.h and protocol.h
ClosedPublic
Actions

Authored by Fabien on Jan 20 2021, 10:25.

Tags

None

Subscribers

Details

Reviewers

PiRK
majcosta

Group Reviewers

Restricted Project

Commits

rABC3297680281f2: tests: Add fuzzing harnesses for functions/classes in chain.h and protocol.h

Summary

Add fuzzing harnesses for functions/classes in chain.h and protocol.h.

Backport of core PR18445.

Adapted to match our codebase.

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 20 2021, 10:25

Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 20 2021, 10:25

Fabien requested review of this revision.Jan 20 2021, 10:25

teamcity edited the summary of this revision. (Show Details)Jan 20 2021, 10:26

Fabien edited the summary of this revision. (Show Details)Jan 20 2021, 10:27

Harbormaster completed remote builds in B14863: Diff 27111.Jan 20 2021, 10:42

Fabien added a child revision: D8981: tests: Add fuzzing harness for functions/classes in flatfile.h, merkleblock.h, random.h, serialize.h and span.h.Jan 20 2021, 11:22

PiRK accepted this revision.Jan 20 2021, 14:50

This revision is now accepted and ready to land.Jan 20 2021, 14:50

majcosta requested changes to this revision.Jan 20 2021, 15:18

majcosta added a subscriber: majcosta.

majcosta added inline comments.

src/test/fuzz/chain.cpp
34 ↗	(On Diff #27111)	looks like GetChainTxCount(), GetChainSize(), UpdateChainStats(), GetHeaderReceivedTime(), GetReceivedTimeDiff() are missing
src/test/fuzz/protocol.cpp
27 ↗	(On Diff #27111)	same for GetKind(), IsTx() and IsSomeBlock()

This revision now requires changes to proceed.Jan 20 2021, 15:18

Add missing methods

Harbormaster completed remote builds in B14875: Diff 27135.Jan 20 2021, 16:15

majcosta accepted this revision.Jan 20 2021, 16:47

This revision is now accepted and ready to land.Jan 20 2021, 16:47

Closed by commit rABC3297680281f2: tests: Add fuzzing harnesses for functions/classes in chain.h and protocol.h (authored by practicalswift <practicalswift@users.noreply.github.com>, committed by Fabien). · Explain WhyJan 20 2021, 19:59

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC3297680281f2: tests: Add fuzzing harnesses for functions/classes in chain.h and protocol.h.

Revision Contents
Changeset List

Path

Size

src/

test/

fuzz/

4 lines

98 lines

37 lines

Diff 27149

src/test/fuzz/CMakeLists.txt

Loading...

src/test/fuzz/chain.cpp

Loading...

src/test/fuzz/protocol.cpp

Loading...