diff options
author | Pedro Alves <palves@redhat.com> | 2017-09-29 13:06:34 +0100 |
---|---|---|
committer | Pedro Alves <palves@redhat.com> | 2017-09-29 13:06:34 +0100 |
commit | 94c207e097a3a30ce71fc8f700325113968c276b (patch) | |
tree | db4cc65cb31bde9bcfbdac32c6ab0be00d540b3e /gdb/gdbserver/linux-low.c | |
parent | f6ac8c52c955b487dcb56d05fc93fced7b203294 (diff) | |
download | gdb-94c207e097a3a30ce71fc8f700325113968c276b.zip gdb-94c207e097a3a30ce71fc8f700325113968c276b.tar.gz gdb-94c207e097a3a30ce71fc8f700325113968c276b.tar.bz2 |
Fix gdbserver regression exposed by gdb.threads/multi-create-ns-info-thr.exp
Commit 8629910955a7 ("Add thread_db_notice_clone to gdbserver")
introduced calls into libthread_db without making sure that the
current thread is pointing to a know-stopped thread. This resulted in
sometimes thread_db_notice_clone failing->find_one_thread failing like
this, as seen when running gdb.threads/multi-create-ns-info-thr.exp:
~~~
Thread <6> executing
Thread <7> executing
gdbserver: PID mismatch! Expected 27472, got 27471
gdbserver: Cannot find thread after clone.
Thread <1000> executing
Thread <1001> executing
~~~
Things go south from here and sometimes that ends up resulting in
gdbserver crashing and the test failing.
gdb/gdbserver/ChangeLog:
2017-09-29 Pedro Alves <palves@redhat.com>
* linux-low.c (handle_extended_wait): Pass parent thread instead
of process to thread_db_notice_clone.
* linux-low.h (thread_db_notice_clone): Replace parent process
parameter with parent thread parameter.
* thread-db.c (find_one_thread): Add comment.
(thread_db_notice_clone): Replace parent process parameter with
parent thread parameter. Temporarily switch to the parent thread.
Diffstat (limited to 'gdb/gdbserver/linux-low.c')
-rw-r--r-- | gdb/gdbserver/linux-low.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gdb/gdbserver/linux-low.c b/gdb/gdbserver/linux-low.c index c62dc19..54005c1 100644 --- a/gdb/gdbserver/linux-low.c +++ b/gdb/gdbserver/linux-low.c @@ -656,7 +656,7 @@ handle_extended_wait (struct lwp_info **orig_event_lwp, int wstat) new_lwp->status_pending = status; } - thread_db_notice_clone (get_thread_process (event_thr), ptid); + thread_db_notice_clone (event_thr, ptid); /* Don't report the event. */ return 1; |