ELF/BFD: Handle both variants of the 64-bit Linux core PRPSINFO note
authorMaciej W. Rozycki <macro@imgtec.com>
Wed, 11 Oct 2017 14:01:41 +0000 (15:01 +0100)
committerMaciej W. Rozycki <macro@imgtec.com>
Wed, 11 Oct 2017 14:01:41 +0000 (15:01 +0100)
Fix commit 70a38d42c5b3 ("New entry points for writing Linux NT_PRPSINFO
notes."), <https://sourceware.org/ml/binutils/2013-02/msg00023.html>,
and handle both variants of the 64-bit Linux core PRPSINFO note across
all targets.

The 64-bit Linux core PRPSINFO note matches the 64-bit kernel structure,
defined as follows:

(gdb) ptype struct elf_prpsinfo
type = struct elf_prpsinfo {
    char pr_state;
    char pr_sname;
    char pr_zomb;
    char pr_nice;
    unsigned long pr_flag;
    __kernel_uid_t pr_uid;
    __kernel_gid_t pr_gid;
    pid_t pr_pid;
    pid_t pr_ppid;
    pid_t pr_pgrp;
    pid_t pr_sid;
    char pr_fname[16];
    char pr_psargs[80];
}
(gdb)

where the individual data types of most members are the same across all
64-bit Linux ports, with the exception of `__kernel_uid_t' and
`__kernel_gid_t'.  These are defined in <asm-generic/posix_types.h> as
32-bit `unsigned int' by default, however overridden as 16-bit `unsigned
short' in port-specific <asm/posix_types.h> for the SuperH target.

The default is already handled, as from the commit referred.  Add the
other variant then and provide a backend flag to switch between the two
cases possible, with the 32-bit one being the default and the 16-bit one
explicitly selected.  Set the flag in the SuperH target backend.  This
arrangement is analogous to how the 32-bit Linux core PRPSINFO note is
handled.

bfd/
* elf-bfd.h (elf_backend_data): Add `linux_prpsinfo64_ugid16'
member.
* elf-linux-core.h (elf_external_linux_prpsinfo64): Rename to...
(elf_external_linux_prpsinfo64_ugid32): ... this.
(swap_linux_prpsinfo32_out): Rename to...
(swap_linux_prpsinfo32_ugid32_out): ... this.
(elf_external_linux_prpsinfo64_ugid16): New structure.
(swap_linux_prpsinfo64_ugid16_out): New function.
* elfxx-target.h [!elf_backend_linux_prpsinfo64_ugid16]
(elf_backend_linux_prpsinfo64_ugid16): Define.
(elfNN_bed): Initialize `linux_prpsinfo64_ugid16' member.
* elf.c (elfcore_write_linux_prpsinfo64): Handle both variants
of the 64-bit Linux core PRPSINFO note.
* elf64-sh64.c (elf_backend_linux_prpsinfo64_ugid16): Define.

bfd/ChangeLog
bfd/elf-bfd.h
bfd/elf-linux-core.h
bfd/elf.c
bfd/elf64-sh64.c
bfd/elfxx-target.h

index c25b281191913c99378879aa82a32e3d640fa49a..eb819200fe2871bf5b7ae4a5b2610002ab75e8b8 100644 (file)
@@ -1,3 +1,20 @@
+2017-10-11  Maciej W. Rozycki  <macro@imgtec.com>
+
+       * elf-bfd.h (elf_backend_data): Add `linux_prpsinfo64_ugid16'
+       member.
+       * elf-linux-core.h (elf_external_linux_prpsinfo64): Rename to...
+       (elf_external_linux_prpsinfo64_ugid32): ... this.
+       (swap_linux_prpsinfo32_out): Rename to...
+       (swap_linux_prpsinfo32_ugid32_out): ... this.
+       (elf_external_linux_prpsinfo64_ugid16): New structure.
+       (swap_linux_prpsinfo64_ugid16_out): New function.
+       * elfxx-target.h [!elf_backend_linux_prpsinfo64_ugid16]
+       (elf_backend_linux_prpsinfo64_ugid16): Define.
+       (elfNN_bed): Initialize `linux_prpsinfo64_ugid16' member.
+       * elf.c (elfcore_write_linux_prpsinfo64): Handle both variants
+       of the 64-bit Linux core PRPSINFO note.
+       * elf64-sh64.c (elf_backend_linux_prpsinfo64_ugid16): Define.
+
 2017-10-11  Maciej W. Rozycki  <macro@imgtec.com>
 
        * elf-bfd.h (elf_backend_data): Add `linux_prpsinfo32_ugid16'
index 4788fd058bde488868923cc40ec05e418071e078..c3560ce4fc62043e0d4cfe1dac71dbd3f4975dbb 100644 (file)
@@ -1561,6 +1561,10 @@ struct elf_backend_data
   /* True if the 32-bit Linux PRPSINFO structure's `pr_uid' and `pr_gid'
      members use a 16-bit data type.  */
   unsigned linux_prpsinfo32_ugid16 : 1;
+
+  /* True if the 64-bit Linux PRPSINFO structure's `pr_uid' and `pr_gid'
+     members use a 16-bit data type.  */
+  unsigned linux_prpsinfo64_ugid16 : 1;
 };
 
 /* Information about reloc sections associated with a bfd_elf_section_data
index 20eef9ab3a72710173cd4f1d2ce0306a43ffb03d..1183be3e2b545cec1a7e828cda963fd2a15b6e88 100644 (file)
@@ -129,10 +129,14 @@ swap_linux_prpsinfo32_ugid16_out
    ABI-defined, thus we choose to use char arrays here in order to
    avoid dealing with different types in different architectures.
 
+   This is the variant for targets which use a 32-bit data type for
+   UID and GID, as most Linux ports do.  The SH64 port uses a 16-bit
+   data type instead; see below for the alternative variant.
+
    This structure will ultimately be written in the corefile's note
    section, as the PRPSINFO.  */
 
-struct elf_external_linux_prpsinfo64
+struct elf_external_linux_prpsinfo64_ugid32
   {
     char pr_state;                     /* Numeric process state.  */
     char pr_sname;                     /* Char for pr_state.  */
@@ -151,12 +155,13 @@ struct elf_external_linux_prpsinfo64
   };
 
 /* Helper function to copy an elf_internal_linux_prpsinfo in host
-   endian to an elf_external_linux_prpsinfo64 in target endian.  */
+   endian to an elf_external_linux_prpsinfo64_ugid32 in target endian.  */
 
 static inline void
-swap_linux_prpsinfo64_out (bfd *obfd,
-                          const struct elf_internal_linux_prpsinfo *from,
-                          struct elf_external_linux_prpsinfo64 *to)
+swap_linux_prpsinfo64_ugid32_out
+  (bfd *obfd,
+   const struct elf_internal_linux_prpsinfo *from,
+   struct elf_external_linux_prpsinfo64_ugid32 *to)
 {
   bfd_put_8 (obfd, from->pr_state, &to->pr_state);
   bfd_put_8 (obfd, from->pr_sname, &to->pr_sname);
@@ -173,4 +178,57 @@ swap_linux_prpsinfo64_out (bfd *obfd,
   strncpy (to->pr_psargs, from->pr_psargs, sizeof (to->pr_psargs));
 }
 
+/* External 64-bit structure for PRPSINFO.  This structure is
+   ABI-defined, thus we choose to use char arrays here in order to
+   avoid dealing with different types in different architectures.
+
+   This is the variant for the SH64 port which uses a 16-bit data
+   type for UID and GID.  Most Linux ports use a 32-bit data type
+   instead; see above for the alternative variant.
+
+   This structure will ultimately be written in the corefile's note
+   section, as the PRPSINFO.  */
+
+struct elf_external_linux_prpsinfo64_ugid16
+  {
+    char pr_state;                     /* Numeric process state.  */
+    char pr_sname;                     /* Char for pr_state.  */
+    char pr_zomb;                      /* Zombie.  */
+    char pr_nice;                      /* Nice val.  */
+    char gap[4];
+    char pr_flag[8];                   /* Flags.  */
+    char pr_uid[2];
+    char pr_gid[2];
+    char pr_pid[4];
+    char pr_ppid[4];
+    char pr_pgrp[4];
+    char pr_sid[4];
+    char pr_fname[16];                 /* Filename of executable.  */
+    char pr_psargs[80];                        /* Initial part of arg list.  */
+  };
+
+/* Helper function to copy an elf_internal_linux_prpsinfo in host
+   endian to an elf_external_linux_prpsinfo64_ugid16 in target endian.  */
+
+static inline void
+swap_linux_prpsinfo64_ugid16_out
+  (bfd *obfd,
+   const struct elf_internal_linux_prpsinfo *from,
+   struct elf_external_linux_prpsinfo64_ugid16 *to)
+{
+  bfd_put_8 (obfd, from->pr_state, &to->pr_state);
+  bfd_put_8 (obfd, from->pr_sname, &to->pr_sname);
+  bfd_put_8 (obfd, from->pr_zomb, &to->pr_zomb);
+  bfd_put_8 (obfd, from->pr_nice, &to->pr_nice);
+  bfd_put_64 (obfd, from->pr_flag, to->pr_flag);
+  bfd_put_16 (obfd, from->pr_uid, to->pr_uid);
+  bfd_put_16 (obfd, from->pr_gid, to->pr_gid);
+  bfd_put_32 (obfd, from->pr_pid, to->pr_pid);
+  bfd_put_32 (obfd, from->pr_ppid, to->pr_ppid);
+  bfd_put_32 (obfd, from->pr_pgrp, to->pr_pgrp);
+  bfd_put_32 (obfd, from->pr_sid, to->pr_sid);
+  strncpy (to->pr_fname, from->pr_fname, sizeof (to->pr_fname));
+  strncpy (to->pr_psargs, from->pr_psargs, sizeof (to->pr_psargs));
+}
+
 #endif
index 4a150de96051a1f4ee3ae4a1f5921036d5b1fd3c..9cecb249221a114d6bcea3e2c6ecf2090abaecc7 100644 (file)
--- a/bfd/elf.c
+++ b/bfd/elf.c
@@ -10489,11 +10489,22 @@ elfcore_write_linux_prpsinfo64
   (bfd *abfd, char *buf, int *bufsiz,
    const struct elf_internal_linux_prpsinfo *prpsinfo)
 {
-  struct elf_external_linux_prpsinfo64 data;
+  if (get_elf_backend_data (abfd)->linux_prpsinfo64_ugid16)
+    {
+      struct elf_external_linux_prpsinfo64_ugid16 data;
 
-  swap_linux_prpsinfo64_out (abfd, prpsinfo, &data);
-  return elfcore_write_note (abfd, buf, bufsiz,
-                            "CORE", NT_PRPSINFO, &data, sizeof (data));
+      swap_linux_prpsinfo64_ugid16_out (abfd, prpsinfo, &data);
+      return elfcore_write_note (abfd, buf, bufsiz,
+                                "CORE", NT_PRPSINFO, &data, sizeof (data));
+    }
+  else
+    {
+      struct elf_external_linux_prpsinfo64_ugid32 data;
+
+      swap_linux_prpsinfo64_ugid32_out (abfd, prpsinfo, &data);
+      return elfcore_write_note (abfd, buf, bufsiz,
+                                "CORE", NT_PRPSINFO, &data, sizeof (data));
+    }
 }
 
 char *
index 1cffb2ec5649f8c71646b6760a24d9f43897ca13..8994e21daffd7246b5a385766bfcb19894e3ac53 100644 (file)
@@ -3944,6 +3944,8 @@ static const struct bfd_elf_special_section sh64_elf64_special_sections[]=
 #define elf_backend_got_header_size    24
 #define elf_backend_dtrel_excludes_plt 1
 
+#define elf_backend_linux_prpsinfo64_ugid16    TRUE
+
 #include "elf64-target.h"
 
 /* NetBSD support.  */
index a8b9770fdd17602ba6315c54c3191e2236c0af7d..ed185e77d2e3ba707601103246fe88dcbbd0941d 100644 (file)
 #ifndef elf_backend_linux_prpsinfo32_ugid16
 #define elf_backend_linux_prpsinfo32_ugid16 FALSE
 #endif
+#ifndef elf_backend_linux_prpsinfo64_ugid16
+#define elf_backend_linux_prpsinfo64_ugid16 FALSE
+#endif
 #ifndef elf_backend_stack_align
 #define elf_backend_stack_align 16
 #endif
@@ -899,7 +902,8 @@ static struct elf_backend_data elfNN_bed =
   elf_backend_caches_rawsize,
   elf_backend_extern_protected_data,
   elf_backend_always_renumber_dynsyms,
-  elf_backend_linux_prpsinfo32_ugid16
+  elf_backend_linux_prpsinfo32_ugid16,
+  elf_backend_linux_prpsinfo64_ugid16
 };
 
 /* Forward declaration for use when initialising alternative_target field.  */