* frame.c (frame_pop): Check to see whether there's a frame to

	which we can pop first.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 2a3985c..3766bc1 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,10 @@
 2007-03-13  Nathan Froyd  <froydnj@codesourcery.com>
 
+       * frame.c (frame_pop): Check to see whether there's a frame to
+       which we can pop first.
+
+2007-03-13  Nathan Froyd  <froydnj@codesourcery.com>
+
        * MAINTAINERS (Write After Approval): Add myself.
 
 2007-03-09  Markus Deuling  <deuling@de.ibm.com>

diff --git a/gdb/frame.c b/gdb/frame.c
index a40bbb2..8a14b29 100644 (file)
--- a/gdb/frame.c
+++ b/gdb/frame.c
@@ -514,13 +514,22 @@ frame_save_as_regcache (struct frame_info *this_frame)
 void
 frame_pop (struct frame_info *this_frame)
 {
+  struct frame_info *prev_frame;
+  struct regcache *scratch;
+  struct cleanup *cleanups;
+
+  /* Ensure that we have a frame to pop to.  */
+  prev_frame = get_prev_frame_1 (this_frame);
+
+  if (!prev_frame)
+    error (_("Cannot pop the initial frame."));
+
   /* Make a copy of all the register values unwound from this frame.
      Save them in a scratch buffer so that there isn't a race between
      trying to extract the old values from the current_regcache while
      at the same time writing new values into that same cache.  */
-  struct regcache *scratch
-    = frame_save_as_regcache (get_prev_frame_1 (this_frame));
-  struct cleanup *cleanups = make_cleanup_regcache_xfree (scratch);
+  scratch = frame_save_as_regcache (prev_frame);
+  cleanups = make_cleanup_regcache_xfree (scratch);
 
   /* FIXME: cagney/2003-03-16: It should be possible to tell the
      target's register cache that it is about to be hit with a burst