Page MenuHomePhabricator
Differential D3466

Merge #10415: [tests] Speed up fuzzing by ~200x when using afl-fuzz
Changes PlannedPublic
Actions

Authored by nakihito on Jun 28 2019, 19:11.

Details

Reviewers
deadalnix
Fabien
jasonbcox
markblundeberg
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Summary

693247b [test] Speed up fuzzing by ~200x when using afl-fuzz (practicalswift)

Tree-SHA512: 95922fc2616b8cb00dd531ed1140a52bbda4e04292dd8c1c60a8f49dbf6ccb797a18b61180b3fb68d695456b478a1f5ae7fda47e8ecee41dd65555487aef40a3

Backport of Core PR10415

Test Plan
make check

Diff Detail

Repository
rABC Bitcoin ABC
Branch
PR10415
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 6548
Build 11143: Bitcoin ABC Buildbot (legacy)
Build 11142: arc lint + arc unit

Event Timeline

nakihito created this revision.Jun 28 2019, 19:11
Owners added a reviewer: Restricted Owners Package.Jun 28 2019, 19:11
Herald added a reviewer: Restricted Project. · View Herald TranscriptJun 28 2019, 19:11
teamcity added a comment.Jun 28 2019, 19:11

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

Harbormaster completed remote builds in B6548: Diff 9790.Jun 28 2019, 19:22
teamcity added a comment.Jun 28 2019, 19:22

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing passed.

Fabien requested changes to this revision.Jun 28 2019, 21:22

Code looks good, but I fail to see how make check is testing that.

This revision now requires changes to proceed.Jun 28 2019, 21:22
nakihito planned changes to this revision.Jun 29 2019, 02:11
In D3466#81930, @Fabien wrote:

Code looks good, but I fail to see how make check is testing that.

You're right. Was under the impression that it was run with make check. Will update test plan later.

Revision Contents
Changeset List

PathSize
doc/
8 lines
src/
test/
23 lines

Diff 9790

View Options

doc/fuzzing.md

Loading...
View Options

src/test/test_bitcoin_fuzzy.cpp

Loading...