aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMaciej W. Rozycki <macro@imgtec.com>2017-10-11 15:01:41 +0100
committerMaciej W. Rozycki <macro@imgtec.com>2017-10-11 15:01:41 +0100
commit3c9a7b0d6d84425ea24d222a1a31e206e587374f (patch)
tree78ec2893e3f1867fe1a45ceac22d45315b81f0f2
parenta2f63b2e7a5da01ff828b45c7e0e57265ee24131 (diff)
downloadgdb-3c9a7b0d6d84425ea24d222a1a31e206e587374f.zip
gdb-3c9a7b0d6d84425ea24d222a1a31e206e587374f.tar.gz
gdb-3c9a7b0d6d84425ea24d222a1a31e206e587374f.tar.bz2
ELF/BFD: Handle both variants of the 64-bit Linux core PRPSINFO note
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.
-rw-r--r--bfd/ChangeLog17
-rw-r--r--bfd/elf-bfd.h4
-rw-r--r--bfd/elf-linux-core.h68
-rw-r--r--bfd/elf.c19
-rw-r--r--bfd/elf64-sh64.c2
-rw-r--r--bfd/elfxx-target.h6
6 files changed, 106 insertions, 10 deletions
diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index c25b281..eb81920 100644
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,5 +1,22 @@
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'
member.
(elfcore_write_ppc_linux_prpsinfo32): Remove prototype.
diff --git a/bfd/elf-bfd.h b/bfd/elf-bfd.h
index 4788fd0..c3560ce 100644
--- a/bfd/elf-bfd.h
+++ b/bfd/elf-bfd.h
@@ -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
diff --git a/bfd/elf-linux-core.h b/bfd/elf-linux-core.h
index 20eef9a..1183be3 100644
--- a/bfd/elf-linux-core.h
+++ b/bfd/elf-linux-core.h
@@ -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
diff --git a/bfd/elf.c b/bfd/elf.c
index 4a150de..9cecb24 100644
--- 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 *
diff --git a/bfd/elf64-sh64.c b/bfd/elf64-sh64.c
index 1cffb2e..8994e21 100644
--- a/bfd/elf64-sh64.c
+++ b/bfd/elf64-sh64.c
@@ -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. */
diff --git a/bfd/elfxx-target.h b/bfd/elfxx-target.h
index a8b9770..ed185e7 100644
--- a/bfd/elfxx-target.h
+++ b/bfd/elfxx-target.h
@@ -135,6 +135,9 @@
#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. */