diff options
| author | Pedro Alves <palves@redhat.com> | 2013-04-19 15:22:14 +0000 |
|---|---|---|
| committer | Pedro Alves <palves@redhat.com> | 2013-04-19 15:22:14 +0000 |
| commit | 463920bf28e3aea7ebac2cf10b8e5b98e56400eb (patch) | |
| tree | 642eea66a0cd31cf0e342df64b6046ca5ffe4c09 | |
| parent | 70242eb16c06c7fcc080003af6c17ad9dfd634c1 (diff) | |
| download | fsf-binutils-gdb-463920bf28e3aea7ebac2cf10b8e5b98e56400eb.zip fsf-binutils-gdb-463920bf28e3aea7ebac2cf10b8e5b98e56400eb.tar.gz fsf-binutils-gdb-463920bf28e3aea7ebac2cf10b8e5b98e56400eb.tar.bz2 | |
ppc-linux-tdep.c: Wrong signness for buffer holding instructions.
There seems to be no reason for this buffer to be signed. We pass it
around to functions expecting it to be unsigned (which triggers
-Wpointer-sign warnings).
gdb/
2013-04-19 Pedro Alves <palves@redhat.com>
* ppc-linux-tdep.c (ppc_skip_trampoline_code): Change local
'insnbuf' buffer type to unsigned int[].
| -rw-r--r-- | gdb/ChangeLog | 5 | ||||
| -rw-r--r-- | gdb/ppc-linux-tdep.c | 2 |
2 files changed, 6 insertions, 1 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 9314749..f71ebc6 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,10 @@ 2013-04-19 Pedro Alves <palves@redhat.com> + * ppc-linux-tdep.c (ppc_skip_trampoline_code): Change local + 'insnbuf' buffer type to unsigned int[]. + +2013-04-19 Pedro Alves <palves@redhat.com> + * mips-tdep.c (mips_read_pc): Change local 'pc' type to LONGEST. 2013-04-19 Pedro Alves <palves@redhat.com> diff --git a/gdb/ppc-linux-tdep.c b/gdb/ppc-linux-tdep.c index 6895562..25b6c0b 100644 --- a/gdb/ppc-linux-tdep.c +++ b/gdb/ppc-linux-tdep.c @@ -355,7 +355,7 @@ powerpc_linux_in_dynsym_resolve_code (CORE_ADDR pc) static CORE_ADDR ppc_skip_trampoline_code (struct frame_info *frame, CORE_ADDR pc) { - int insnbuf[POWERPC32_PLT_STUB_LEN]; + unsigned int insnbuf[POWERPC32_PLT_STUB_LEN]; struct gdbarch *gdbarch = get_frame_arch (frame); struct gdbarch_tdep *tdep = gdbarch_tdep (gdbarch); enum bfd_endian byte_order = gdbarch_byte_order (gdbarch); |