[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH-for-5.1 1/2] configure: do not clobber CFLAGS with --enable-f
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH-for-5.1 1/2] configure: do not clobber CFLAGS with --enable-fuzzing |
Date: |
Thu, 9 Jul 2020 07:01:14 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.5.0 |
On 7/8/20 10:01 PM, Alexander Bulekov wrote:
> When configuring with --enable-fuzzing, we overwrote the CFLAGS
> added by all the preceding checks. Instead of overwriting CFLAGS, append
> the ones we need.
>
> Fixes: adc28027ff ("fuzz: add configure flag --enable-fuzzing")
> Reported-by: Li Qiang <liq3ea@163.com>
> Signed-off-by: Alexander Bulekov <alxndr@bu.edu>
> ---
> configure | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/configure b/configure
> index ee6c3c6792..078ebaa9f3 100755
> --- a/configure
> +++ b/configure
> @@ -7898,7 +7898,7 @@ if test "$fuzzing" = "yes" ; then
> if test "$have_fuzzer" = "yes"; then
> FUZZ_LDFLAGS=" -fsanitize=address,fuzzer"
> FUZZ_CFLAGS=" -fsanitize=address,fuzzer"
> - CFLAGS=" -fsanitize=address,fuzzer-no-link"
> + CFLAGS="$CFLAGS -fsanitize=address,fuzzer-no-link"
> else
> error_exit "Your compiler doesn't support -fsanitize=address,fuzzer"
> exit 1
>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>