diff options
| -rw-r--r-- | ChangeLog | 6 | ||||
| -rw-r--r-- | nis/nis_table.c | 9 | ||||
| -rw-r--r-- | stdio-common/vfscanf.c | 10 |
3 files changed, 25 insertions, 0 deletions
diff --git a/ChangeLog b/ChangeLog index f1c6ac5cd0..4df1d8c52e 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +2016-01-20 Chris Metcalf <cmetcalf@ezchip.com> + + * nis/nis_table.c (__follow_path): Disable diagnostic for + uninitialized variable that is a false positive for gcc 4.7. + * stdio-common/vfscanf.c (_IO_vfwscanf): Likewise. + 2016-01-20 Roland McGrath <roland@hack.frob.com> * sysdeps/nacl/lowlevellock-futex.h diff --git a/nis/nis_table.c b/nis/nis_table.c index 2831164a19..1d607a49c4 100644 --- a/nis/nis_table.c +++ b/nis/nis_table.c @@ -19,6 +19,7 @@ #include <assert.h> #include <string.h> #include <rpcsvc/nis.h> +#include <libc-internal.h> #include "nis_xdr.h" #include "nis_intern.h" @@ -175,6 +176,12 @@ __follow_path (char **tablepath, char **tableptr, struct ib_request *ibreq, *tableptr = *tablepath; } + + /* Since tableptr is only set here, and it's set when tablepath is NULL, + which it is initially defined as, we know it will always be set here. */ + DIAG_PUSH_NEEDS_COMMENT; + DIAG_IGNORE_NEEDS_COMMENT (4.7, "-Wmaybe-uninitialized"); + if (*tableptr == NULL) return NIS_NOTFOUND; @@ -182,6 +189,8 @@ __follow_path (char **tablepath, char **tableptr, struct ib_request *ibreq, if (newname[0] == '\0') return NIS_NOTFOUND; + DIAG_POP_NEEDS_COMMENT; + newname = strdup (newname); if (newname == NULL) return NIS_NOMEMORY; diff --git a/stdio-common/vfscanf.c b/stdio-common/vfscanf.c index f8f97cd1d2..8cd59557a6 100644 --- a/stdio-common/vfscanf.c +++ b/stdio-common/vfscanf.c @@ -27,6 +27,7 @@ #include <string.h> #include <wchar.h> #include <wctype.h> +#include <libc-internal.h> #include <libc-lock.h> #include <locale/localeinfo.h> #include <scratch_buffer.h> @@ -1535,6 +1536,13 @@ _IO_vfscanf_internal (_IO_FILE *s, const char *format, _IO_va_list argptr, { /* Get the string for the digits with value N. */ #ifdef COMPILE_WSCANF + + /* wcdigits_extended[] is fully set in the loop + above, but the test for "map != NULL" is done + inside the loop here and outside the loop there. */ + DIAG_PUSH_NEEDS_COMMENT; + DIAG_IGNORE_NEEDS_COMMENT (4.7, "-Wmaybe-uninitialized"); + if (__glibc_unlikely (map != NULL)) wcdigits[n] = wcdigits_extended[n]; else @@ -1542,6 +1550,8 @@ _IO_vfscanf_internal (_IO_FILE *s, const char *format, _IO_va_list argptr, _NL_CURRENT (LC_CTYPE, _NL_CTYPE_INDIGITS0_WC + n); wcdigits[n] += from_level; + DIAG_POP_NEEDS_COMMENT; + if (c == (wint_t) *wcdigits[n]) { to_level = from_level; |