Skip to content

Commit f71a3a8

Browse files
author
MarcoFalke
committed
Merge #18939: doc: add c++17-enable flag to fuzzing instructions
872aa25 doc: add c++17-enable to fuzzing instructions (Martin Zumsande) Pull request description: Update the fuzzing doc because after the merge of #18901, C++17 is required for compilation. ACKs for top commit: practicalswift: ACK 872aa25 MarcoFalke: ACK 872aa25 Tree-SHA512: 47e37c033690de1d1fa644bf0cebb256036b32a5784021cc0d3b32e6188822d7f517d4342990dc7ec98de6d650794aeb85483157e69e141d6bd011993e124575
2 parents ec4d27f + 872aa25 commit f71a3a8

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

doc/fuzzing.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ To quickly get started fuzzing Bitcoin Core using [libFuzzer](https://llvm.org/d
88
$ git clone https://github.com/bitcoin/bitcoin
99
$ cd bitcoin/
1010
$ ./autogen.sh
11-
$ CC=clang CXX=clang++ ./configure --enable-fuzz --with-sanitizers=address,fuzzer,undefined
11+
$ CC=clang CXX=clang++ ./configure --enable-fuzz --with-sanitizers=address,fuzzer,undefined --enable-c++17
1212
# macOS users: If you have problem with this step then make sure to read "macOS hints for
1313
# libFuzzer" on https://github.com/bitcoin/bitcoin/blob/master/doc/fuzzing.md#macos-hints-for-libfuzzer
1414
$ make
@@ -103,7 +103,7 @@ You may also need to take care of giving the correct path for `clang` and
103103
Full configure that was tested on macOS Catalina with `brew` installed `llvm`:
104104
105105
```sh
106-
./configure --enable-fuzz --with-sanitizers=fuzzer,address,undefined CC=/usr/local/opt/llvm/bin/clang CXX=/usr/local/opt/llvm/bin/clang++ --disable-asm
106+
./configure --enable-fuzz --with-sanitizers=fuzzer,address,undefined CC=/usr/local/opt/llvm/bin/clang CXX=/usr/local/opt/llvm/bin/clang++ --disable-asm --enable-c++17
107107
```
108108
109109
Read the [libFuzzer documentation](https://llvm.org/docs/LibFuzzer.html) for more information. This [libFuzzer tutorial](https://github.com/google/fuzzing/blob/master/tutorial/libFuzzerTutorial.md) might also be of interest.
@@ -121,7 +121,7 @@ $ git clone https://github.com/google/afl
121121
$ make -C afl/
122122
$ make -C afl/llvm_mode/
123123
$ ./autogen.sh
124-
$ CC=$(pwd)/afl/afl-clang-fast CXX=$(pwd)/afl/afl-clang-fast++ ./configure --enable-fuzz
124+
$ CC=$(pwd)/afl/afl-clang-fast CXX=$(pwd)/afl/afl-clang-fast++ ./configure --enable-fuzz --enable-c++17
125125
$ make
126126
# For macOS you may need to ignore x86 compilation checks when running "make". If so,
127127
# try compiling using: AFL_NO_X86=1 make
@@ -148,7 +148,7 @@ $ git clone https://github.com/google/honggfuzz
148148
$ cd honggfuzz/
149149
$ make
150150
$ cd ..
151-
$ CC=$(pwd)/honggfuzz/hfuzz_cc/hfuzz-clang CXX=$(pwd)/honggfuzz/hfuzz_cc/hfuzz-clang++ ./configure --enable-fuzz --with-sanitizers=address,undefined
151+
$ CC=$(pwd)/honggfuzz/hfuzz_cc/hfuzz-clang CXX=$(pwd)/honggfuzz/hfuzz_cc/hfuzz-clang++ ./configure --enable-fuzz --with-sanitizers=address,undefined --enable-c++17
152152
$ make
153153
$ mkdir -p inputs/
154154
$ honggfuzz/honggfuzz -i inputs/ -- src/test/fuzz/process_message

0 commit comments

Comments
 (0)