Page MenuHomePhabricator

build: Allow to configure --with-sanitizers=fuzzer
AcceptedPublic

Authored by Fabien on Mon, Dec 2, 14:34.

Details

Reviewers
deadalnix
jasonbcox
Group Reviewers
Restricted Project
Summary
Test Plan
./configure --disable-ccache --with-sanitizers=fuzzer,address CC=clang CXX=clang++
make

Note: make fails to link on my machines (also tested on core), I don't
know what they used for testing. This will be fixed by a later PR so
it's not a big issue.

Diff Detail

Repository
rABC Bitcoin ABC
Branch
PR15047
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 8371
Build 14761: Bitcoin ABC Buildbot
Build 14760: arc lint + arc unit

Event Timeline

Fabien created this revision.Mon, Dec 2, 14:34
Herald added a reviewer: Restricted Project. · View Herald TranscriptMon, Dec 2, 14:34
deadalnix added inline comments.Mon, Dec 2, 14:37
doc/fuzzing.md
1

Out of the scope for this patch, but it looks like something that should be updated.

Fabien added inline comments.Mon, Dec 2, 14:40
doc/fuzzing.md
1

Yes, all the fuzzy-related files need update. I'm gonna do them all in one after I'm done with all the backports (there are not plenty) in order to avoid conflicts.

jasonbcox accepted this revision.Tue, Dec 3, 23:20
jasonbcox added a subscriber: jasonbcox.

PS: I think your test plan indentation is broken.

This revision is now accepted and ready to land.Tue, Dec 3, 23:20
Fabien edited the test plan for this revision. (Show Details)Wed, Dec 4, 07:08