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 <liq...@163.com>
> Signed-off-by: Alexander Bulekov <alx...@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é <phi...@redhat.com>
