[BZ #20271] Add newlines in __libc_fatal calls.

diff --git a/ChangeLog b/ChangeLog
index e4a1b51d2ec7a761b8ea666052f110e9b95346fc..d11440bc0fe4db07d4f287335902fb0f65db552d 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,25 @@
+2018-08-31  Paul Pluzhnikov  <ppluzhnikov@google.com>
+
+       [BZ #20271]
+       * include/stdio.h (__libc_fatal): Mention newline in comment.
+       * grp/initgroups.c (internal_getgrouplist): Add missing newline.
+       * nptl/pthread_cond_wait.c (__pthread_cond_wait_common): Likewise.
+       * nscd/initgrcache.c (addinitgroupsX): Likewise.
+       * nss/nsswitch.c (__nss_next2): Likewise.
+       * sysdeps/aarch64/dl-irel.h (elf_irela): Likewise.
+       * sysdeps/arm/dl-irel.h (elf_irel): Likewise.
+       * sysdeps/generic/unwind-dw2.c (execute_cfa_program): Likewise.
+       * sysdeps/i386/dl-irel.h (elf_irel): Likewise.
+       * sysdeps/powerpc/powerpc32/dl-irel.h (elf_irel): Likewise.
+       * sysdeps/powerpc/powerpc64/dl-irel.h (elf_irel): Likewise.
+       * sysdeps/s390/dl-irel.h (elf_irel): Likewise.
+       * sysdeps/sparc/sparc32/dl-irel.h (elf_irel): Likewise.
+       * sysdeps/sparc/sparc64/dl-irel.h (elf_irel): Likewise.
+       * sysdeps/x86_64/dl-irel.h (elf_irel): Likewise.
+       * sysdeps/nptl/futex-internal.h (futex_wake): Likewise.
+       * sysdeps/unix/sysv/linux/netlink_assert_response.c
+       (__netlink_assert_response): Likewise.
+
 2018-08-31  Joseph Myers  <joseph@codesourcery.com>
 
        * conform/glibcconform.py: New file.

diff --git a/grp/initgroups.c b/grp/initgroups.c
index f056fbf5aa6aa14c49dea8d7553f9f0ed1f72f41..93e7f5814da6286dd8cc5efbb823b2594a1f09ca 100644 (file)
--- a/grp/initgroups.c
+++ b/grp/initgroups.c
@@ -128,7 +128,7 @@ internal_getgrouplist (const char *user, gid_t group, long int *size,
 
       /* This is really only for debugging.  */
       if (NSS_STATUS_TRYAGAIN > status || status > NSS_STATUS_RETURN)
-       __libc_fatal ("illegal status in internal_getgrouplist");
+       __libc_fatal ("Illegal status in internal_getgrouplist.\n");
 
       /* For compatibility reason we will continue to look for more
         entries using the next service even though data has already

diff --git a/include/stdio.h b/include/stdio.h
index 49383fb2d70814b8285466652c22b10e8660d9ae..51ada4a4c4b53034d26dee04bccfd6a829db275b 100644 (file)
--- a/include/stdio.h
+++ b/include/stdio.h
@@ -98,7 +98,8 @@ enum __libc_message_action
   do_backtrace = 1 << 1        /* Backtrace.  */
 };
 
-/* Print out MESSAGE on the error output and abort.  */
+/* Print out MESSAGE (which should end with a newline) on the error output
+   and abort.  */
 extern void __libc_fatal (const char *__message)
      __attribute__ ((__noreturn__));
 extern void __libc_message (enum __libc_message_action action,

diff --git a/nptl/pthread_cond_wait.c b/nptl/pthread_cond_wait.c
index 3e1105418210288ef21c37730a59dc8bc619ab3b..ebf07ca82d87de7db9c9ada0e391c7bb42131363 100644 (file)
--- a/nptl/pthread_cond_wait.c
+++ b/nptl/pthread_cond_wait.c
@@ -516,7 +516,7 @@ __pthread_cond_wait_common (pthread_cond_t *cond, pthread_mutex_t *mutex,
                  struct timespec rt;
                  if (__clock_gettime (CLOCK_MONOTONIC, &rt) != 0)
                    __libc_fatal ("clock_gettime does not support "
-                                 "CLOCK_MONOTONIC");
+                                 "CLOCK_MONOTONIC\n");
                  /* Convert the absolute timeout value to a relative
                     timeout.  */
                  rt.tv_sec = abstime->tv_sec - rt.tv_sec;

diff --git a/nscd/initgrcache.c b/nscd/initgrcache.c
index 2c74951f579f4afd2bef2609685d0a765b5fcc76..4764f14a45f68e0a183315812931a08bb947b34b 100644 (file)
--- a/nscd/initgrcache.c
+++ b/nscd/initgrcache.c
@@ -159,7 +159,7 @@ addinitgroupsX (struct database_dyn *db, int fd, request_header *req,
 
       /* This is really only for debugging.  */
       if (NSS_STATUS_TRYAGAIN > status || status > NSS_STATUS_RETURN)
-       __libc_fatal ("illegal status in internal_getgrouplist");
+       __libc_fatal ("Illegal status in internal_getgrouplist.\n");
 
       any_success |= status == NSS_STATUS_SUCCESS;
 

diff --git a/nss/nsswitch.c b/nss/nsswitch.c
index ee46f24424bc1ca2085f4fd7f1060ae330ee5bae..3c48b4b85e881cdbc9cdea6ee3fdc9fc3d21bcd1 100644 (file)
--- a/nss/nsswitch.c
+++ b/nss/nsswitch.c
@@ -235,7 +235,7 @@ __nss_next2 (service_user **ni, const char *fct_name, const char *fct2_name,
       /* This is really only for debugging.  */
       if (__builtin_expect (NSS_STATUS_TRYAGAIN > status
                            || status > NSS_STATUS_RETURN, 0))
-        __libc_fatal ("illegal status in __nss_next");
+        __libc_fatal ("Illegal status in __nss_next.\n");
 
        if (nss_next_action (*ni, status) == NSS_ACTION_RETURN)
         return 1;

diff --git a/sysdeps/aarch64/dl-irel.h b/sysdeps/aarch64/dl-irel.h
index 5889ee187b7a1eafc22bbf54502557482955475b..bef71ed0f31a6387c4baa4574da18e9e5d21c0ff 100644 (file)
--- a/sysdeps/aarch64/dl-irel.h
+++ b/sysdeps/aarch64/dl-irel.h
@@ -47,7 +47,7 @@ elf_irela (const ElfW(Rela) *reloc)
       *reloc_addr = value;
     }
   else
-    __libc_fatal ("unexpected reloc type in static binary");
+    __libc_fatal ("Unexpected reloc type in static binary.\n");
 }
 
 #endif

diff --git a/sysdeps/arm/dl-irel.h b/sysdeps/arm/dl-irel.h
index a7b6456075659baf945bcf2c7643f14ff78db250..be6eb7743eb5f08d577f8851b3203528ec9c89c3 100644 (file)
--- a/sysdeps/arm/dl-irel.h
+++ b/sysdeps/arm/dl-irel.h
@@ -46,7 +46,7 @@ elf_irel (const Elf32_Rel *reloc)
       *reloc_addr = value;
     }
   else
-    __libc_fatal ("unexpected reloc type in static binary");
+    __libc_fatal ("Unexpected reloc type in static binary.\n");
 }
 
 #endif /* dl-irel.h */

diff --git a/sysdeps/generic/unwind-dw2.c b/sysdeps/generic/unwind-dw2.c
index 082609b34a3f773bc9caa45d4e84708d7407c4ac..724c16a7f0bf465b2139f665f0db4932678c28dd 100644 (file)
--- a/sysdeps/generic/unwind-dw2.c
+++ b/sysdeps/generic/unwind-dw2.c
@@ -843,7 +843,7 @@ execute_cfa_program (const unsigned char *insn_ptr,
            struct frame_state_reg_info *old_rs = fs->regs.prev;
 #ifdef _LIBC
            if (old_rs == NULL)
-             __libc_fatal ("invalid DWARF unwind data");
+             __libc_fatal ("Invalid DWARF unwind data.\n");
            else
 #endif
              {

diff --git a/sysdeps/i386/dl-irel.h b/sysdeps/i386/dl-irel.h
index 55303180c7aca4957525e31c04dc869c796b1507..bcaf0668acf8e2f20e6de0e10d91e4ff2b1b2548 100644 (file)
--- a/sysdeps/i386/dl-irel.h
+++ b/sysdeps/i386/dl-irel.h
@@ -45,7 +45,7 @@ elf_irel (const Elf32_Rel *reloc)
       *reloc_addr = value;
     }
   else
-    __libc_fatal ("unexpected reloc type in static binary");
+    __libc_fatal ("Unexpected reloc type in static binary.\n");
 }
 
 #endif /* dl-irel.h */

diff --git a/sysdeps/nptl/futex-internal.h b/sysdeps/nptl/futex-internal.h
index 1a5624789d4ab117f4c1ab7544168b508401ab9f..6fd27f0df6c27b69e3bada224dd9bf081eb23289 100644 (file)
--- a/sysdeps/nptl/futex-internal.h
+++ b/sysdeps/nptl/futex-internal.h
@@ -197,7 +197,7 @@ futex_wake (unsigned int* futex_word, int processes_to_wake, int private);
 static __always_inline __attribute__ ((__noreturn__)) void
 futex_fatal_error (void)
 {
-  __libc_fatal ("The futex facility returned an unexpected error code.");
+  __libc_fatal ("The futex facility returned an unexpected error code.\n");
 }
 
 #endif  /* futex-internal.h */

diff --git a/sysdeps/powerpc/powerpc32/dl-irel.h b/sysdeps/powerpc/powerpc32/dl-irel.h
index a7368b25829618cb3dee9a87fdf9f8eda21fa7c3..61d0e4cf61ec45d30538ffc3f4867aed794a36db 100644 (file)
--- a/sysdeps/powerpc/powerpc32/dl-irel.h
+++ b/sysdeps/powerpc/powerpc32/dl-irel.h
@@ -46,7 +46,7 @@ elf_irela (const Elf32_Rela *reloc)
       *reloc_addr = value;
     }
   else
-    __libc_fatal ("unexpected reloc type in static binary");
+    __libc_fatal ("Unexpected reloc type in static binary.\n");
 }
 
 #endif /* dl-irel.h */

diff --git a/sysdeps/powerpc/powerpc64/dl-irel.h b/sysdeps/powerpc/powerpc64/dl-irel.h
index ab13c04358868270b2426d064a6b060e4e9e0ac5..2fd0ee8a86e85ba0cd1d42bb3c1d90828ae82708 100644 (file)
--- a/sysdeps/powerpc/powerpc64/dl-irel.h
+++ b/sysdeps/powerpc/powerpc64/dl-irel.h
@@ -57,7 +57,7 @@ elf_irela (const Elf64_Rela *reloc)
 #endif
     }
   else
-    __libc_fatal ("unexpected reloc type in static binary");
+    __libc_fatal ("Unexpected reloc type in static binary.\n");
 }
 
 #endif /* dl-irel.h */

diff --git a/sysdeps/s390/dl-irel.h b/sysdeps/s390/dl-irel.h
index d8ba7ba42709f45ca97436d30a5c8b853a2f8842..ecb24f0a9be0daa7005a436e1a97ae147152458f 100644 (file)
--- a/sysdeps/s390/dl-irel.h
+++ b/sysdeps/s390/dl-irel.h
@@ -46,7 +46,7 @@ elf_irela (const ElfW(Rela) *reloc)
       *reloc_addr = value;
     }
   else
-    __libc_fatal ("unexpected reloc type in static binary");
+    __libc_fatal ("Unexpected reloc type in static binary.\n");
 }
 
 #endif /* dl-irel.h */

diff --git a/sysdeps/sparc/sparc32/dl-irel.h b/sysdeps/sparc/sparc32/dl-irel.h
index ffca36864f24d1fba869ef3bfae6ac0b640ed2f3..cf47cda8345b1a399ee9c5c6970d1556169c09c7 100644 (file)
--- a/sysdeps/sparc/sparc32/dl-irel.h
+++ b/sysdeps/sparc/sparc32/dl-irel.h
@@ -56,7 +56,7 @@ elf_irela (const Elf32_Rela *reloc)
   else if (r_type == R_SPARC_NONE)
     ;
   else
-    __libc_fatal ("unexpected reloc type in static binary");
+    __libc_fatal ("Unexpected reloc type in static binary.\n");
 }
 
 #endif /* dl-irel.h */

diff --git a/sysdeps/sparc/sparc64/dl-irel.h b/sysdeps/sparc/sparc64/dl-irel.h
index c5cd3057aca1baf64438d07f0e765fd837ffa2b1..446fed18365cfd13abf9e0119d4e704d98143949 100644 (file)
--- a/sysdeps/sparc/sparc64/dl-irel.h
+++ b/sysdeps/sparc/sparc64/dl-irel.h
@@ -59,7 +59,7 @@ elf_irela (const Elf64_Rela *reloc)
   else if (r_type == R_SPARC_NONE)
     ;
   else
-    __libc_fatal ("unexpected reloc type in static binary");
+    __libc_fatal ("Unexpected reloc type in static binary.\n");
 }
 
 #endif /* dl-irel.h */

diff --git a/sysdeps/unix/sysv/linux/netlink_assert_response.c b/sysdeps/unix/sysv/linux/netlink_assert_response.c
index f31ccb52ffa564363e116639de5b334513228e0a..6afc3a17ced18e1c3df904484d0b9174ec736f26 100644 (file)
--- a/sysdeps/unix/sysv/linux/netlink_assert_response.c
+++ b/sysdeps/unix/sysv/linux/netlink_assert_response.c
@@ -72,12 +72,12 @@ __netlink_assert_response (int fd, ssize_t result)
           char message[200];
           if (family < 0)
             __snprintf (message, sizeof (message),
-                        "Unexpected error %d on netlink descriptor %d",
+                        "Unexpected error %d on netlink descriptor %d.\n",
                         error_code, fd);
           else
             __snprintf (message, sizeof (message),
                         "Unexpected error %d on netlink descriptor %d"
-                        " (address family %d)",
+                        " (address family %d).\n",
                         error_code, fd, family);
           __libc_fatal (message);
         }

diff --git a/sysdeps/x86_64/dl-irel.h b/sysdeps/x86_64/dl-irel.h
index 6ecc50fb42333c19519b94a0652888117d64717c..33f100d8b1781ea775b3f10273c416949d142a65 100644 (file)
--- a/sysdeps/x86_64/dl-irel.h
+++ b/sysdeps/x86_64/dl-irel.h
@@ -45,7 +45,7 @@ elf_irela (const ElfW(Rela) *reloc)
       *reloc_addr = value;
     }
   else
-    __libc_fatal ("unexpected reloc type in static binary");
+    __libc_fatal ("Unexpected reloc type in static binary.\n");
 }
 
 #endif /* dl-irel.h */