* frame.c (find_saved_register): Avoid a NULL pointer

dereference and actually walk the frame list.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 043cf342c95492acc9d1e992fd32eed10ea0a5d8..4bc72c60dbc89e2d2f8a401a782c793fc0a6af50 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2002-04-20  Jason Thorpe  <thorpej@wasabisystems.com>
+
+       * frame.c (find_saved_register): Avoid a NULL pointer
+       dereference and actually walk the frame list.
+
 2002-04-20  Andrew Cagney  <ac131313@redhat.com>
 
        * gdbarch.sh (gdbarch_update_p): Keep the list of architectures

diff --git a/gdb/frame.c b/gdb/frame.c
index 7923660e1ab03848bc8e3138e326fa324f0be71a..819ee2cb0175a21d1103bd426df37cc99b5929f2 100644 (file)
--- a/gdb/frame.c
+++ b/gdb/frame.c
@@ -83,9 +83,10 @@ find_saved_register (struct frame_info *frame, int regnum)
   while (1)
     {
       QUIT;
-      frame1 = get_next_frame (frame1);
-      if (frame1 == 0 || frame1 == frame)
+      frame1 = get_next_frame (frame);
+      if (frame1 == 0)
        break;
+      frame = frame1;
       FRAME_INIT_SAVED_REGS (frame1);
       if (frame1->saved_regs[regnum])
        addr = frame1->saved_regs[regnum];