Fix erroroneous use of '%zu' in elfcore_grok_win32pstatus
authorJon Turney <jon.turney@dronecode.org.uk>
Fri, 21 Aug 2020 15:30:00 +0000 (16:30 +0100)
committerJon Turney <jon.turney@dronecode.org.uk>
Mon, 24 Aug 2020 17:04:55 +0000 (18:04 +0100)
As reported in [1], _bfd_error_handler() doesn't support '%zu'.

module_name_size is always 32-bits in the data structure we are
extracting it from, so use an unsigned int to store it instead.

[1] https://sourceware.org/pipermail/gdb-patches/2020-August/171391.html

bfd/ChangeLog:

2020-08-21  Jon Turney  <jon.turney@dronecode.org.uk>

* elf.c (elfcore_grok_win32pstatus): Change name_size to unsigned
int. Use '%u' format with  _bfd_error_handler to render it.

bfd/ChangeLog
bfd/elf.c

index ca79c4e583072bfd0fdaefe3232a000e89598324..f2d18122bbc6c2b4eaf7804fbc471fa8c7b5535e 100644 (file)
@@ -1,3 +1,8 @@
+2020-08-21  Jon Turney  <jon.turney@dronecode.org.uk>
+
+       * elf.c (elfcore_grok_win32pstatus): Change name_size to unsigned
+       int. Use '%u' format with  _bfd_error_handler to render it.
+
 2020-08-25  Alan Modra  <amodra@gmail.com>
 
        PR 26489
index ecd9217b34d237a92f7eeb3989fe2c5ed007f9c6..f32118ad404a7e455d975df3c180a0bfec6959f3 100644 (file)
--- a/bfd/elf.c
+++ b/bfd/elf.c
@@ -10140,7 +10140,7 @@ elfcore_grok_win32pstatus (bfd *abfd, Elf_Internal_Note *note)
   char buf[30];
   char *name;
   size_t len;
-  size_t name_size;
+  unsigned int name_size;
   asection *sect;
   unsigned int type;
   int is_active_thread;
@@ -10248,7 +10248,7 @@ elfcore_grok_win32pstatus (bfd *abfd, Elf_Internal_Note *note)
 
       if (note->descsz < 12 + name_size)
         {
-          _bfd_error_handler (_("%pB: win32pstatus NOTE_INFO_MODULE of size %lu is too small to contain a name of size %zu"),
+          _bfd_error_handler (_("%pB: win32pstatus NOTE_INFO_MODULE of size %lu is too small to contain a name of size %u"),
                               abfd, note->descsz, name_size);
           return TRUE;
         }