Merge pull request FRRouting#5127 from opensourcerouting/asan-updates

configure.ac/doc: update address sanitizer pointers
sudhanshukumar22 · Oct 9, 2019 · a4862a0 · a4862a0
2 parents 61fcca7 + 4260f16
commit a4862a0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 7 deletions.
diff --git a/configure.ac b/configure.ac
@@ -328,8 +328,8 @@ if test "$enable_thread_sanitizer" = "yes"; then
   ])
 fi
 if test "$enable_memory_sanitizer" = "yes"; then
-  AC_C_FLAG([-fsanitize=thread -fPIE -pie], [
-    AC_MSG_ERROR([$CC does not support Thread Sanitizer.])
+  AC_C_FLAG([-fsanitize=memory -fPIE -pie], [
+    AC_MSG_ERROR([$CC does not support Memory Sanitizer.])
   ], [
     SAN_FLAGS="-fsanitize=memory -fPIE -pie"
   ])

diff --git a/doc/developer/topotests.rst b/doc/developer/topotests.rst
@@ -189,11 +189,8 @@ for ``master`` branch:
    git clone https://github.com/FRRouting/frr.git
    cd frr
    ./bootstrap.sh
-   export CC=gcc
-   export CFLAGS="-O1 -g -fsanitize=address -fno-omit-frame-pointer"
-   export LD=gcc
-   export LDFLAGS="-g -fsanitize=address -ldl"
-   ./configure --enable-shared=no \
+   ./configure \
+       --enable-address-sanitizer \
        --prefix=/usr/lib/frr --sysconfdir=/etc/frr \
        --localstatedir=/var/run/frr \
        --sbindir=/usr/lib/frr --bindir=/usr/lib/frr \