summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPedro Alves <pedro@codesourcery.com>2008-11-20 13:23:25 +0000
committerPedro Alves <pedro@codesourcery.com>2008-11-20 13:23:25 +0000
commit9ed3dd4e6ae59ae7e8c9a46c6e1deb54f4dad10d (patch)
tree1e281e9376599210a693690fa4fe4e5d70f4ad4e
parentc49abb1626e43d04cb67e34d11ec8e0b81a863dd (diff)
downloadgdb-9ed3dd4e6ae59ae7e8c9a46c6e1deb54f4dad10d.tar.gz
* infrun.c (resume): If following a fork, reread the current
thread. Avoid dereferencing a possibly dangling pointer.
-rw-r--r--gdb/ChangeLog5
-rw-r--r--gdb/infrun.c8
2 files changed, 11 insertions, 2 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 4eabe76f447..62b98fe5f99 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2008-11-20 Pedro Alves <pedro@codesourcery.com>
+
+ * infrun.c (resume): If following a fork, reread the current
+ thread. Avoid dereferencing a possibly dangling pointer.
+
2008-11-19 Doug Evans <dje@google.com>
* inferior.h (proceed_to_finish): Delete, unused.
diff --git a/gdb/infrun.c b/gdb/infrun.c
index f218fa0e1f9..93ee00bc00a 100644
--- a/gdb/infrun.c
+++ b/gdb/infrun.c
@@ -1053,6 +1053,10 @@ a command like `return' or `jump' to continue execution."));
pending_follow.kind = TARGET_WAITKIND_SPURIOUS;
if (follow_fork ())
should_resume = 0;
+
+ /* Following a child fork will change our notion of current
+ thread. */
+ tp = inferior_thread ();
break;
case TARGET_WAITKIND_EXECD:
@@ -1148,11 +1152,11 @@ a command like `return' or `jump' to continue execution."));
displaced_step_dump_bytes (gdb_stdlog, buf, sizeof (buf));
}
- target_resume (resume_ptid, step, sig);
-
/* Avoid confusing the next resume, if the next stop/resume
happens to apply to another thread. */
tp->stop_signal = TARGET_SIGNAL_0;
+
+ target_resume (resume_ptid, step, sig);
}
discard_cleanups (old_cleanups);