diff options
author | Paul Eggert <eggert@cs.ucla.edu> | 2023-04-24 12:06:55 -0700 |
---|---|---|
committer | Paul Eggert <eggert@cs.ucla.edu> | 2023-04-24 12:07:40 -0700 |
commit | 4bb14d69493fd3bbc328b86755420892a88d3fd8 (patch) | |
tree | fc09f7a7eefc2cc077dc4b9bb4ce77961da52669 /ChangeLog | |
parent | b98993a1baaa2fc39b301676ecbd8bb29e1d9c96 (diff) | |
download | gnulib-4bb14d69493fd3bbc328b86755420892a88d3fd8.tar.gz |
fclose: pacify gcc -Wanalyzer-file-leak
Without this patch, building coreutils with
--enable-gcc-checking=expensive would fail with a message like
“lib/exclude.c:682:6: error: leak of FILE 'in' [CWE-775]
[-Werror=analyzer-file-leak]”, because Gnulib replaced fclose but
not fopen, and GCC saw a call to fopen followed by a call to
rpl_fclose. The patch causes GCC to instead see a call to
rpl_fopen followed by rpl_fclose.
* m4/fclose.m4 (gl_FUNC_FCLOSE): Replace fopen when replacing
fclose.
* modules/fclose (Depends-on): Add fopen.
(configure.ac): Replace fopen when replacing fclose.
Diffstat (limited to 'ChangeLog')
-rw-r--r-- | ChangeLog | 13 |
1 files changed, 13 insertions, 0 deletions
@@ -1,5 +1,18 @@ 2023-04-24 Paul Eggert <eggert@cs.ucla.edu> + fclose: pacify gcc -Wanalyzer-file-leak + Without this patch, building coreutils with + --enable-gcc-checking=expensive would fail with a message like + “lib/exclude.c:682:6: error: leak of FILE 'in' [CWE-775] + [-Werror=analyzer-file-leak]”, because Gnulib replaced fclose but + not fopen, and GCC saw a call to fopen followed by a call to + rpl_fclose. The patch causes GCC to instead see a call to + rpl_fopen followed by rpl_fclose. + * m4/fclose.m4 (gl_FUNC_FCLOSE): Replace fopen when replacing + fclose. + * modules/fclose (Depends-on): Add fopen. + (configure.ac): Replace fopen when replacing fclose. + unsetenv-tests: pacify -Wanalyzer-putenv-of-auto-var * tests/test-unsetenv.c (main): Make entry static. Even with this change, it’s unclear whether this test is portable POSIX code, but |