On 4/18/23 10:26 AM, PGNet Dev wrote:
I'm building nginx mainline v1.24 on Fedora.

on F37, with gcc 12,

     gcc --version
         gcc (GCC) 12.2.1 20221121 (Red Hat 12.2.1-4)
         Copyright (C) 2022 Free Software Foundation, Inc.
        This is free software; see the source for copying conditions. There is NO         warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.


build's good.

Upgrading to today's new/latest F38, with gcc 13,

     gcc --version
         gcc (GCC) 13.0.1 20230401 (Red Hat 13.0.1-0)
         Copyright (C) 2023 Free Software Foundation, Inc.
        This is free software; see the source for copying conditions. There is NO         warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

builds for target F38+ fail @ dangling-pointer errors,

     ...
     src/njs_iterator.c: In function 'njs_object_iterate':
    src/njs_iterator.c:358:25: error: storing the address of local variable 'string_obj' in '*args.value' [-Werror=dangling-pointer=]
       358 |             args->value = &string_obj;
           |             ~~~~~~~~~~~~^~~~~~~~~~~~~
     ...
     cc1: all warnings being treated as errors

adding

     -Wno-dangling-pointer

to build flags worksaround it, with successful build.

for ref,

FAILED build log:

     
https://download.copr.fedorainfracloud.org/results/pgfed/nginx-mainline/fedora-38-x86_64/05802768-nginx/build.log.gz

OK build log:

     
https://download.copr.fedorainfracloud.org/results/pgfed/nginx-mainline/fedora-38-x86_64/05802814-nginx/build.log.gz

I'm checking to see whether the error flag was added to GCC 13 upstream, or just to Redhat/Fedora flags ...

GCC 13 is not released yet, right?

Can you reproduce the issue with -Wdangling-pointer option and GCC 12?


_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
https://mailman.nginx.org/mailman/listinfo/nginx-devel
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
https://mailman.nginx.org/mailman/listinfo/nginx-devel

Reply via email to