Differential D9062

tests: Don't limit fuzzing inputs to 1 MB for afl-fuzz (now: ∞ ∀ fuzzers)
ClosedPublic
Actions

Authored by Fabien on Jan 26 2021, 12:41.

Details

Reviewers

majcosta

Group Reviewers

Restricted Project

Commits

rABCd87c0e81140a: tests: Don't limit fuzzing inputs to 1 MB for afl-fuzz (now: ∞ ∀ fuzzers)

Summary

This change provides a level playing field for all fuzzers which allows
for fair benchmarking using projects such as the excellent FuzzBench
project.

Prior to this commit we limited afl-fuzz to ≤1 MB inputs but allowed
unlimited length inputs for all other fuzzers.

Backport of core PR19052.

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 26 2021, 12:41

Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 26 2021, 12:41

Fabien requested review of this revision.Jan 26 2021, 12:41

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

Harbormaster completed remote builds in B14989: Diff 27321.Jan 26 2021, 13:33

majcosta accepted this revision.Jan 26 2021, 13:52

This revision is now accepted and ready to land.Jan 26 2021, 13:52

Closed by commit rABCd87c0e81140a: tests: Don't limit fuzzing inputs to 1 MB for afl-fuzz (now: ∞ ∀ fuzzers) (authored by practicalswift <practicalswift@users.noreply.github.com>, committed by Fabien). · Explain WhyJan 26 2021, 15:16

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABCd87c0e81140a: tests: Don't limit fuzzing inputs to 1 MB for afl-fuzz (now: ∞ ∀ fuzzers).

Revision Contents
Changeset List

Path

Size

src/

test/

fuzz/

fuzz.cpp

4 lines

Diff 27324

View Options

tests: Don't limit fuzzing inputs to 1 MB for afl-fuzz (now: ∞ ∀ fuzzers)ClosedPublicActions