[PATCH] configure.ac: disable -fcf-protection

[Austin English]

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=48161
Signed-off-by: Austin English <austinenglish(a)gmail.com>
---
 configure.ac | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/configure.ac b/configure.ac
index 7f2c3cda23..942585adfe 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2113,6 +2113,9 @@ then
     CFLAGS="$CFLAGS -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=0"
 fi
 
+dnl **** Disable fcf-protection, it breaks a ton of apps
+WINE_TRY_CFLAGS([-fcf-protection=none])
+
 dnl **** Check for CFI directives support ****
 
 AC_CACHE_CHECK([whether CFI directives are supported in assembly code], ac_cv_c_cfi_support,
-- 
2.23.0

[Zebediah Figura]

On 12/6/19 7:33 AM, Austin English wrote:
> Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=48161
> Signed-off-by: Austin English <austinenglish(a)gmail.com>
> ---
>  configure.ac | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/configure.ac b/configure.ac
> index 7f2c3cda23..942585adfe 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -2113,6 +2113,9 @@ then
>      CFLAGS="$CFLAGS -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=0"
>  fi
>
> +dnl **** Disable fcf-protection, it breaks a ton of apps
> +WINE_TRY_CFLAGS([-fcf-protection=none])
> +
>  dnl **** Check for CFI directives support ****
>
> AC_CACHE_CHECK([whether CFI directives are supported in assembly code], ac_cv_c_cfi_support,

Unfortunately this won't help, since WINE_TRY_CFLAGS puts the requested
options into EXTRACFLAGS, but that'll be overriden later by the option
in CFLAGS. We need that option to be removed from the build script.