Skip to content

Commit

Permalink
oss-fuzz: turn off fuzz-introspector
Browse files Browse the repository at this point in the history
fuzz-introspector passes -fuse-ld=gold and -flto using CFLAGS/LDFLAGS and due to
mesonbuild/meson#6377 (comment) and
mesonbuild/meson#6377 it doesn't mix well with meson.
It's possible to build systemd with duct tape there using something like
google/oss-fuzz#7583 (comment) but
apparently even with gold and lto some parts of systemd are missing from
reports (presumably due to google/oss-fuzz#7598).
Let's just fail here for now to make it clear that fuzz-introspector isn't supported.
  • Loading branch information
evverx committed Apr 21, 2022
1 parent 255689a commit 7f12323
Showing 1 changed file with 12 additions and 0 deletions.
12 changes: 12 additions & 0 deletions tools/oss-fuzz.sh
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,18 @@ else
CFLAGS="$CFLAGS $UBSAN_FLAGS"
CXXFLAGS="$CXXFLAGS $UBSAN_FLAGS"
fi

if [[ "$SANITIZER" == introspector ]]; then
# fuzz-introspector passes -fuse-ld=gold and -flto using CFLAGS/LDFLAGS and due to
# https://github.com/mesonbuild/meson/issues/6377#issuecomment-575977919 and
# https://github.com/mesonbuild/meson/issues/6377 it doesn't mix well with meson.
# It's possible to build systemd with duct tape there using something like
# https://github.com/google/oss-fuzz/pull/7583#issuecomment-1104011067 but
# apparently even with gold and lto some parts of systemd are missing from
# reports (presumably due to https://github.com/google/oss-fuzz/issues/7598).
# Let's just fail here for now to make it clear that fuzz-introspector isn't supported.
exit 1
fi
fi

if ! meson "$build" "-D$fuzzflag" -Db_lundef=false; then
Expand Down

0 comments on commit 7f12323

Please sign in to comment.