diff options
| author | mtg0001 <106211915+mtg0001@users.noreply.github.com> | 2026-08-04 15:31:55 -0300 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2026-08-04 15:31:55 -0300 |
| commit | b3a5a858e536acf65e6c6761293626628f2227bf (patch) | |
| tree | 881dbe44ac6c0ea28f5a65cbfd9b835ba73ea648 /.github/workflows | |
| parent | 173b4649e90f4f5ba49e92569eb063709e647c7d (diff) | |
fuzz: add P2P message parser harness (#142)
* fuzz: add P2P message parser harness
* fuzz: avoid duplicate legacy header includes
* fuzz: seed the corpus with our network magic, not Bitcoin's
The two seed inputs began f5 be b6 d9, which is Bitcoin's message
start. Ours is bf 20 5c fd. CMessageHeader::IsValid rejects on that
field first, so both seeds died at the first check and the smoke run
exercised the reject path and nothing else.
libFuzzer would have to mutate its way onto four exact bytes before
reaching any parser, which does not happen in 128 runs. The job would
have gone green while covering none of what it was added to cover.
---------
Co-authored-by: Gilcimar Duarte <ti@gpm.infra>
Co-authored-by: Bitflash-sh <btf01@proton.me>
Diffstat (limited to '.github/workflows')
| -rw-r--r-- | .github/workflows/ci.yml | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 985ba8a..9efd66f 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml | |||
| @@ -25,6 +25,9 @@ jobs: | |||
| 25 | - name: Run Linux selftests | 25 | - name: Run Linux selftests |
| 26 | run: make tests | 26 | run: make tests |
| 27 | 27 | ||
| 28 | - name: Run P2P message fuzz smoke | ||
| 29 | run: make fuzz-net-message-smoke | ||
| 30 | |||
| 28 | windows-ucrt64: | 31 | windows-ucrt64: |
| 29 | name: Windows UCRT64 selftests | 32 | name: Windows UCRT64 selftests |
| 30 | runs-on: windows-latest | 33 | runs-on: windows-latest |