]> Savannah Git Hosting - gnulib.git/commitdiff
Sync up error.c with glibc
authorSiddhesh Poyarekar <siddhesh@redhat.com>
Wed, 9 Jul 2014 09:56:17 +0000 (15:26 +0530)
committerSiddhesh Poyarekar <siddhesh@redhat.com>
Thu, 10 Jul 2014 08:24:39 +0000 (13:54 +0530)
This is a partial sync-up of error.c with glibc.  The following
changes went in:

- Get rid of the INTUSE macro
- Remove unused macro ALLOCA_LIMIT
- Fix a potential buffer overflow in error_tail (sourceware bz #15672)
- Fix a potential NULL dereference in strcmp

        error: Sync from glibc master
        * lib/error.c [_LIBC]: Remove INTUSE usage.
        (error_tail): Remove unused macro ALLOCA_LIMIT.
        Fix potential buffer overflow.  Fix potential NULL dereference
        in strcmp.

ChangeLog
lib/error.c

index 66fd9c83485ed1539907bff24f1c3abd27f2ec60..c310fedcb5649c334cf98ad4bef6082a878b1e73 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2014-07-10  Siddhesh Poyarekar  <siddhesh@redhat.com>
+
+       error: Sync from glibc master
+       * lib/error.c [_LIBC]: Remove INTUSE usage.
+       (error_tail): Remove unused macro ALLOCA_LIMIT.
+       Fix potential buffer overflow.  Fix potential NULL dereference
+       in strcmp.
+
 2014-07-09  Pavel Hrdina  <phrdina@redhat.com> (tiny change)
 
        nl_langinfo: fix build under mingw
index 31109df3b5fcf5ba6f7bec6d00d63d5bea87b489..9a3525a8829aca9d1de80d8e5d07827fec4e239b 100644 (file)
@@ -77,9 +77,9 @@ extern void __error_at_line (int status, int errnum, const char *file_name,
 # define error_at_line __error_at_line
 
 # include <libio/iolibio.h>
-# define fflush(s) INTUSE(_IO_fflush) (s)
+# define fflush(s) _IO_fflush (s)
 # undef putc
-# define putc(c, fp) INTUSE(_IO_putc) (c, fp)
+# define putc(c, fp) _IO_putc (c, fp)
 
 # include <bits/libc-lock.h>
 
@@ -201,7 +201,6 @@ error_tail (int status, int errnum, const char *message, va_list args)
 #if _LIBC
   if (_IO_fwide (stderr, 0) > 0)
     {
-# define ALLOCA_LIMIT 2000
       size_t len = strlen (message) + 1;
       wchar_t *wmessage = NULL;
       mbstate_t st;
@@ -237,7 +236,7 @@ error_tail (int status, int errnum, const char *message, va_list args)
           if (res != len)
             break;
 
-          if (__builtin_expect (len >= SIZE_MAX / 2, 0))
+          if (__builtin_expect (len >= SIZE_MAX / sizeof (wchar_t) / 2, 0))
             {
               /* This really should not happen if everything is fine.  */
               res = (size_t) -1;
@@ -342,7 +341,10 @@ error_at_line (int status, int errnum, const char *file_name,
 
       if (old_line_number == line_number
           && (file_name == old_file_name
-              || strcmp (old_file_name, file_name) == 0))
+              || (old_file_name != NULL
+                  && file_name != NULL
+                  && strcmp (old_file_name, file_name) == 0)))
+
         /* Simply return and print nothing.  */
         return;