Don't write to inferior_ptid in tracefile-tfile.c

gdb/ChangeLog:
2020-06-18  Pedro Alves  <palves@redhat.com>

	* tracefile-tfile.c (tfile_target_open): Don't write to
	inferior_ptid directly, instead switch to added thread.
	(tfile_target::close): Use switch_to_no_thread instead of writing
	to inferior_ptid directly.
This commit is contained in:
Pedro Alves 2020-06-18 21:28:23 +01:00
parent 7fb43e53d5
commit 087e161b3c
2 changed files with 11 additions and 3 deletions

View File

@ -1,3 +1,10 @@
2020-06-18 Pedro Alves <palves@redhat.com>
* tracefile-tfile.c (tfile_target_open): Don't write to
inferior_ptid directly, instead switch to added thread.
(tfile_target::close): Use switch_to_no_thread instead of writing
to inferior_ptid directly.
2020-06-18 Pedro Alves <palves@redhat.com> 2020-06-18 Pedro Alves <palves@redhat.com>
* procfs.c (procfs_target::attach): Don't write to inferior_ptid. * procfs.c (procfs_target::attach): Don't write to inferior_ptid.

View File

@ -556,8 +556,9 @@ tfile_target_open (const char *arg, int from_tty)
} }
inferior_appeared (current_inferior (), TFILE_PID); inferior_appeared (current_inferior (), TFILE_PID);
inferior_ptid = ptid_t (TFILE_PID);
add_thread_silent (&tfile_ops, inferior_ptid); thread_info *thr = add_thread_silent (&tfile_ops, ptid_t (TFILE_PID));
switch_to_thread (thr);
if (ts->traceframe_count <= 0) if (ts->traceframe_count <= 0)
warning (_("No traceframes present in this file.")); warning (_("No traceframes present in this file."));
@ -618,7 +619,7 @@ tfile_target::close ()
{ {
gdb_assert (trace_fd != -1); gdb_assert (trace_fd != -1);
inferior_ptid = null_ptid; /* Avoid confusion from thread stuff. */ switch_to_no_thread (); /* Avoid confusion from thread stuff. */
exit_inferior_silent (current_inferior ()); exit_inferior_silent (current_inferior ());
::close (trace_fd); ::close (trace_fd);