[PATCH] fix gdb_disassemble_info

[PATCH] fix gdb_disassemble_info

[Kei Sakamoto]

[-- Attachment #1: Type: text/plain, Size: 413 bytes --]

The attached patch fixes "gdb_disassemble_info" in "disasm.c".

Some targets (m32r, xstormy16, etc.) uses "info->arch" at
"print_insn_*" in "opcodes/*-dis.c". "info" is initialized by
"gdb_disassemble_info" but it does not set "info->arch". So
these targets sometimes abort during dissassemble.


2003-07-04  Kei Sakamoto  <sakamoto.kei@renesas.com>

        * disasm.c (gdb_disassemble_info): Initilize di.arch.

[-- Attachment #2: disasm.c.patch --]
[-- Type: application/octet-stream, Size: 510 bytes --]

--- insight-20030703.orig/src/gdb/disasm.c	Fri May 23 08:00:15 2003
+++ insight-20030703/src/gdb/disasm.c	Fri Jul  4 09:17:15 2003
@@ -328,6 +328,7 @@
      Further, it has been supperseeded by trust-read-only-sections
      (although that should be superseeded by target_trust..._p()).  */
   di.read_memory_func = dis_asm_read_memory;
+  di.arch = gdbarch_bfd_arch_info (gdbarch)->arch;
   di.mach = gdbarch_bfd_arch_info (gdbarch)->mach;
   di.endian = gdbarch_byte_order (gdbarch);
   return di;

Re: [PATCH] fix gdb_disassemble_info

[Andrew Cagney]

> The attached patch fixes "gdb_disassemble_info" in "disasm.c".
> 
> Some targets (m32r, xstormy16, etc.) uses "info->arch" at
> "print_insn_*" in "opcodes/*-dis.c". "info" is initialized by
> "gdb_disassemble_info" but it does not set "info->arch". So
> these targets sometimes abort during dissassemble.
> 
> 
> 2003-07-04  Kei Sakamoto  <sakamoto.kei@renesas.com>
> 
>         * disasm.c (gdb_disassemble_info): Initilize di.arch.

Oops!  I've checked this in - mainline and 6.0 branch.

Andrew