Skip to content

Commit

Permalink
Don't use -fsanitize=fuzzer-no-link with centipede
Browse files Browse the repository at this point in the history
It's pointless and has these drawbacks:
1. It add useless instrumentation not used by centipede that slows
down the target
2. It adds instrumentation that expects an implementation at link
time (__sancov_lowest_stack) which can break projects builds.

Fixes #9609
Related: #9299 #9583
  • Loading branch information
jonathanmetzman committed Feb 7, 2023
1 parent d101ed5 commit 413ba4f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion infra/base-images/base-builder/compile
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ then
fi

# Don't need coverage instrumentation for engine-less, afl++ builds.
if [ $FUZZING_ENGINE = "none" ] || [ $FUZZING_ENGINE = "afl" ]; then
if [ $FUZZING_ENGINE = "none" ] || [ $FUZZING_ENGINE = "afl" ] || [ $FUZZING_ENGINE = "centipede" ] ; then
export COVERAGE_FLAGS=
fi

Expand Down

0 comments on commit 413ba4f

Please sign in to comment.