Differential D9549

Add txrequest fuzz tests
ClosedPublic
Actions

Authored by Fabien on May 18 2021, 15:32.

Details

Reviewers

majcosta

Group Reviewers

Restricted Project

Maniphest Tasks

Restricted Maniphest Task

Commits

rABCececd2475c1c: Add txrequest fuzz tests

Summary

This adds a fuzz test that reimplements a naive reimplementation of
TxRequestTracker (with up to 16 fixed peers and 16 fixed txhashes),
and compares the real implementation against it.

Partial backport of core#19988:
https://github.com/bitcoin/bitcoin/pull/19988/commits/5b03121d60527a193a84c339151481f9c9c1962b

Depends on D9548.

Test Plan

ninja bitcoin-fuzzers

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.May 18 2021, 15:32

Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 18 2021, 15:32

Fabien requested review of this revision.May 18 2021, 15:32

teamcity edited the summary of this revision. (Show Details)May 18 2021, 15:32

Harbormaster completed remote builds in B15755: Diff 28529.May 18 2021, 15:45

Fabien added a child revision: D9550: Change transaction request logic to use txrequest.May 19 2021, 08:36

majcosta accepted this revision.May 20 2021, 03:56

This revision is now accepted and ready to land.May 20 2021, 03:56

Closed by commit rABCececd2475c1c: Add txrequest fuzz tests (authored by Pieter Wuille <pieter@wuille.net>, committed by Fabien). · Explain WhyMay 20 2021, 15:09

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABCececd2475c1c: Add txrequest fuzz tests.

Mengerian added a task: Restricted Maniphest Task.May 21 2021, 15:59

Revision Contents
Changeset List

Path

Size

src/

test/

fuzz/

CMakeLists.txt

1 line

txrequest.cpp

400 lines

Diff 28548

View Options

src/test/fuzz/CMakeLists.txt