This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [Patch H8/300] : Fix for crashing of gdb with" info f" command



******************CHANGELOG TEXT*******************************************************************
2004-06-18  Sherry Samuel  <SherryS@KPITCummins.com>

* h8300-tdep.c (h8300_frame_init_saved_regs) : Check whether frame base is null.

Are you sure this patch is against 6.1 or a current GDB? The function h8300_frame_init_saved_regs has a void return type.


Also, you might want to read over:
  Post GDB 6.2, require new frame code
  http://sources.redhat.com/ml/gdb/2004-06/msg00199.html
As this will affect the h8/300.

Andrew


--- gdb-6.1/gdb/h8300-tdep.c.old 2004-06-17 15:57:18.000000000 +0530
+++ gdb-6.1/gdb/h8300-tdep.c 2004-06-17 17:08:54.000000000 +0530
@@ -500,8 +500,8 @@ h8300_frame_init_saved_regs (struct fram
CORE_ADDR limit = (sal.end && sal.end < get_frame_pc (fi))
? sal.end : get_frame_pc (fi);
/* This will fill in fields in fi. */
-
-
+ if(!get_frame_base (fi))//check whether frame is null.
+ return null_frame_id;//if null return
h8300_examine_prologue (func_addr, limit, get_frame_base (fi),
deprecated_get_frame_saved_regs (fi), fi);



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]