google / oss-fuzz

OSS-Fuzz - continuous fuzzing for open source software.
https://google.github.io/oss-fuzz
Apache License 2.0
10.12k stars 2.15k forks source link

knot-dns: Use latest builder #12095

Closed maflcko closed 1 week ago

github-actions[bot] commented 1 week ago

maflcko has previously contributed to projects/knot-dns. The previous PR was #11714

maflcko commented 1 week ago

An alternative would be to disable the flag that caused this (and is now enabled by default in clang):


diff --git a/projects/knot-dns/build.sh b/projects/knot-dns/build.sh
index 626e91dc9..b0f64b6fe 100755
--- a/projects/knot-dns/build.sh
+++ b/projects/knot-dns/build.sh
@@ -33,6 +33,8 @@ make install

 GNUTLS_CONFIGURE_FLAGS=""
 NETTLE_CONFIGURE_FLAGS=""
+CFLAGS="$CFLAGS     -fno-sanitize-address-use-odr-indicator"
+CXXFLAGS="$CXXFLAGS -fno-sanitize-address-use-odr-indicator"
 if [[ $CFLAGS = *sanitize=memory* ]]; then
   GNUTLS_CONFIGURE_FLAGS="--disable-hardware-acceleration"
   NETTLE_CONFIGURE_FLAGS="--disable-assembler --disable-fat"
jonathanmetzman commented 1 week ago

@jfoote do you approve?

jfoote commented 1 week ago

👍 I haven't been keeping up with knot-dns but this LGTM