]> granicus.if.org Git - sudo/commitdiff
Use -fstack-protector-strong in preference to -fstack-protector-all
authorTodd C. Miller <Todd.Miller@courtesan.com>
Mon, 30 Dec 2013 15:49:34 +0000 (08:49 -0700)
committerTodd C. Miller <Todd.Miller@courtesan.com>
Mon, 30 Dec 2013 15:49:34 +0000 (08:49 -0700)
or -fstack-protector.

configure
configure.ac

index 11183931daa1e3834d625a4c4af72d585f97e0a9..5d97dad85b6eba4de2421e7d04afd6f09b897eb3 100755 (executable)
--- a/configure
+++ b/configure
 
 if test "$enable_hardening" != "no"; then
     if test -n "$GCC"; then
-       { $as_echo "$as_me:${as_lineno-$LINENO}: checking whether C compiler accepts -fstack-protector-all" >&5
+       { $as_echo "$as_me:${as_lineno-$LINENO}: checking whether C compiler accepts -fstack-protector-strong" >&5
+$as_echo_n "checking whether C compiler accepts -fstack-protector-strong... " >&6; }
+if ${ax_cv_check_cflags___fstack_protector_strong+:} false; then :
+  $as_echo_n "(cached) " >&6
+else
+
+  ax_check_save_flags=$CFLAGS
+  CFLAGS="$CFLAGS  -fstack-protector-strong"
+  cat confdefs.h - <<_ACEOF >conftest.$ac_ext
+/* end confdefs.h.  */
+
+int
+main ()
+{
+
+  ;
+  return 0;
+}
+_ACEOF
+if ac_fn_c_try_compile "$LINENO"; then :
+  ax_cv_check_cflags___fstack_protector_strong=yes
+else
+  ax_cv_check_cflags___fstack_protector_strong=no
+fi
+rm -f core conftest.err conftest.$ac_objext conftest.$ac_ext
+  CFLAGS=$ax_check_save_flags
+fi
+{ $as_echo "$as_me:${as_lineno-$LINENO}: result: $ax_cv_check_cflags___fstack_protector_strong" >&5
+$as_echo "$ax_cv_check_cflags___fstack_protector_strong" >&6; }
+if test x"$ax_cv_check_cflags___fstack_protector_strong" = xyes; then :
+
+           { $as_echo "$as_me:${as_lineno-$LINENO}: checking whether the linker accepts -fstack-protector-strong" >&5
+$as_echo_n "checking whether the linker accepts -fstack-protector-strong... " >&6; }
+if ${ax_cv_check_ldflags___fstack_protector_strong+:} false; then :
+  $as_echo_n "(cached) " >&6
+else
+
+  ax_check_save_flags=$LDFLAGS
+  LDFLAGS="$LDFLAGS  -fstack-protector-strong"
+  cat confdefs.h - <<_ACEOF >conftest.$ac_ext
+/* end confdefs.h.  */
+
+int
+main ()
+{
+
+  ;
+  return 0;
+}
+_ACEOF
+if ac_fn_c_try_link "$LINENO"; then :
+  ax_cv_check_ldflags___fstack_protector_strong=yes
+else
+  ax_cv_check_ldflags___fstack_protector_strong=no
+fi
+rm -f core conftest.err conftest.$ac_objext \
+    conftest$ac_exeext conftest.$ac_ext
+  LDFLAGS=$ax_check_save_flags
+fi
+{ $as_echo "$as_me:${as_lineno-$LINENO}: result: $ax_cv_check_ldflags___fstack_protector_strong" >&5
+$as_echo "$ax_cv_check_ldflags___fstack_protector_strong" >&6; }
+if test x"$ax_cv_check_ldflags___fstack_protector_strong" = xyes; then :
+
+               SSP_CFLAGS="-fstack-protector-strong"
+               SSP_LDFLAGS="-Wc,-fstack-protector-strong"
+
+else
+  :
+fi
+
+
+else
+  :
+fi
+
+       if test -z "$SSP_CFLAGS"; then
+           { $as_echo "$as_me:${as_lineno-$LINENO}: checking whether C compiler accepts -fstack-protector-all" >&5
 $as_echo_n "checking whether C compiler accepts -fstack-protector-all... " >&6; }
 if ${ax_cv_check_cflags___fstack_protector_all+:} false; then :
   $as_echo_n "(cached) " >&6
@@ -21792,7 +21868,7 @@ fi
 $as_echo "$ax_cv_check_cflags___fstack_protector_all" >&6; }
 if test x"$ax_cv_check_cflags___fstack_protector_all" = xyes; then :
 
-           { $as_echo "$as_me:${as_lineno-$LINENO}: checking whether the linker accepts -fstack-protector-all" >&5
+               { $as_echo "$as_me:${as_lineno-$LINENO}: checking whether the linker accepts -fstack-protector-all" >&5
 $as_echo_n "checking whether the linker accepts -fstack-protector-all... " >&6; }
 if ${ax_cv_check_ldflags___fstack_protector_all+:} false; then :
   $as_echo_n "(cached) " >&6
@@ -21824,8 +21900,8 @@ fi
 $as_echo "$ax_cv_check_ldflags___fstack_protector_all" >&6; }
 if test x"$ax_cv_check_ldflags___fstack_protector_all" = xyes; then :
 
-               SSP_CFLAGS="-fstack-protector-all"
-               SSP_LDFLAGS="-Wc,-fstack-protector-all"
+                   SSP_CFLAGS="-fstack-protector-all"
+                   SSP_LDFLAGS="-Wc,-fstack-protector-all"
 
 else
   :
@@ -21836,8 +21912,8 @@ else
   :
 fi
 
-       if test -z "$SSP_CFLAGS"; then
-           { $as_echo "$as_me:${as_lineno-$LINENO}: checking whether C compiler accepts -fstack-protector" >&5
+           if test -z "$SSP_CFLAGS"; then
+               { $as_echo "$as_me:${as_lineno-$LINENO}: checking whether C compiler accepts -fstack-protector" >&5
 $as_echo_n "checking whether C compiler accepts -fstack-protector... " >&6; }
 if ${ax_cv_check_cflags___fstack_protector+:} false; then :
   $as_echo_n "(cached) " >&6
@@ -21868,7 +21944,7 @@ fi
 $as_echo "$ax_cv_check_cflags___fstack_protector" >&6; }
 if test x"$ax_cv_check_cflags___fstack_protector" = xyes; then :
 
-               { $as_echo "$as_me:${as_lineno-$LINENO}: checking whether the linker accepts -fstack-protector" >&5
+                   { $as_echo "$as_me:${as_lineno-$LINENO}: checking whether the linker accepts -fstack-protector" >&5
 $as_echo_n "checking whether the linker accepts -fstack-protector... " >&6; }
 if ${ax_cv_check_ldflags___fstack_protector+:} false; then :
   $as_echo_n "(cached) " >&6
@@ -21900,8 +21976,8 @@ fi
 $as_echo "$ax_cv_check_ldflags___fstack_protector" >&6; }
 if test x"$ax_cv_check_ldflags___fstack_protector" = xyes; then :
 
-                   SSP_CFLAGS="-fstack-protector"
-                   SSP_LDFLAGS="-Wc,-fstack-protector"
+                       SSP_CFLAGS="-fstack-protector"
+                       SSP_LDFLAGS="-Wc,-fstack-protector"
 
 else
   :
@@ -21912,6 +21988,7 @@ else
   :
 fi
 
+           fi
        fi
     fi
     { $as_echo "$as_me:${as_lineno-$LINENO}: checking whether the linker accepts -Wl,-z,relro" >&5
index 22d70a647cef36a2774dfe26aa1af7843366d7ad..1c0fecf7a9c03a666170f21cf61bd8920bf21673 100644 (file)
@@ -3575,19 +3575,27 @@ dnl This test relies on AC_LANG_WERROR
 dnl
 if test "$enable_hardening" != "no"; then
     if test -n "$GCC"; then
-       AX_CHECK_COMPILE_FLAG([-fstack-protector-all], [
-           AX_CHECK_LINK_FLAG([-fstack-protector-all], [
-               SSP_CFLAGS="-fstack-protector-all"
-               SSP_LDFLAGS="-Wc,-fstack-protector-all"
+       AX_CHECK_COMPILE_FLAG([-fstack-protector-strong], [
+           AX_CHECK_LINK_FLAG([-fstack-protector-strong], [
+               SSP_CFLAGS="-fstack-protector-strong"
+               SSP_LDFLAGS="-Wc,-fstack-protector-strong"
            ])
        ])
        if test -z "$SSP_CFLAGS"; then
-           AX_CHECK_COMPILE_FLAG([-fstack-protector], [
-               AX_CHECK_LINK_FLAG([-fstack-protector], [
-                   SSP_CFLAGS="-fstack-protector"
-                   SSP_LDFLAGS="-Wc,-fstack-protector"
+           AX_CHECK_COMPILE_FLAG([-fstack-protector-all], [
+               AX_CHECK_LINK_FLAG([-fstack-protector-all], [
+                   SSP_CFLAGS="-fstack-protector-all"
+                   SSP_LDFLAGS="-Wc,-fstack-protector-all"
                ])
            ])
+           if test -z "$SSP_CFLAGS"; then
+               AX_CHECK_COMPILE_FLAG([-fstack-protector], [
+                   AX_CHECK_LINK_FLAG([-fstack-protector], [
+                       SSP_CFLAGS="-fstack-protector"
+                       SSP_LDFLAGS="-Wc,-fstack-protector"
+                   ])
+               ])
+           fi
        fi
     fi
     AX_CHECK_LINK_FLAG([-Wl,-z,relro], [LDFLAGS="${LDFLAGS} -Wl,-z,relro"])