Differential D5484

fuzz: Link BasicTestingSetup (shared with unit tests)
ClosedPublic
Actions

Authored by Fabien on Mar 16 2020, 19:48.

Tags

None

Subscribers

None

Details

Reviewers

Group Reviewers

Restricted Project

Commits

rSTAGING937ea822e740: fuzz: Link BasicTestingSetup (shared with unit tests)
rABC937ea822e740: fuzz: Link BasicTestingSetup (shared with unit tests)

Summary

Completes backport of core PR15504:
https://github.com/bitcoin/bitcoin/pull/15504/commits/faa9b88199c382c04e3aaa2b63479f9066ce8f9c

Depends on D5481 and D4629.

Test Plan

export CC=clang CXX=clang++
../configure --enable-fuzz --with-sanitizers=fuzzer,address \
  --disable-wallet \
  --disable-bench \
  --with-utils=no \
  --with-daemon=no \
  --with-libs=no \
  --with-gui=no \
  --with-seeder=no
make
mkdir -p test/fuzz
cp ../test/fuzz/test_runner.py test/fuzz/
./test/fuzz/test_runner.py -l DEBUG <path_to_corpus>

cmake -GNinja .. -DENABLE_SANITIZERS="address;fuzzer" \
  -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++
ninja bitcoin-fuzzers link-fuzz-test_runner.py
./test/fuzz/test_runner.py -l DEBUG <path_to_corpus>

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Mar 16 2020, 19:48

Herald added a reviewer: Restricted Project. · View Herald TranscriptMar 16 2020, 19:48

[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.

Build Bitcoin-ABC / Diffs / Diff Testing (build-diff) started.

Build Bitcoin-ABC / Diffs / Diff Testing (build-diff) passed.

Harbormaster completed remote builds in B9828: Diff 16954.Mar 16 2020, 20:00

Add a comment to remember to make the test_bitcoin.cpp a library (avoiding relative path).

Build Bitcoin-ABC / Diffs / Diff Testing (build-diff) started.

Build Bitcoin-ABC / Diffs / Diff Testing (build-diff) passed.

Harbormaster completed remote builds in B9841: Diff 16969.Mar 17 2020, 10:18

Fabien added a child revision: D5493: test: Use test_bitcoin setup in bench.Mar 17 2020, 11:08

jasonbcox accepted this revision.Mar 17 2020, 20:18

This revision is now accepted and ready to land.Mar 17 2020, 20:18

Closed by commit rABC937ea822e740: fuzz: Link BasicTestingSetup (shared with unit tests) (authored by MarcoFalke <falke.marco@gmail.com>, committed by Fabien). · Explain WhyMar 19 2020, 15:56

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC937ea822e740: fuzz: Link BasicTestingSetup (shared with unit tests).

Fabien added a commit: rSTAGING937ea822e740: fuzz: Link BasicTestingSetup (shared with unit tests).Apr 7 2020, 23:23

Revision Contents
Changeset List

Path

Size

src/

Makefile.test.include

291 lines

test/

fuzz/

4 lines

2 lines

Diff 17045

src/Makefile.test.include

Loading...

src/test/fuzz/CMakeLists.txt

Loading...

src/test/fuzz/fuzz.h

Loading...