FreeChainXenon/binutils-gdb
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix usage of inferior_ptid in two thread_alive implementations

Browse source 
While inspecting some target code, I noticed that in these two
implementations of thread_alive, inferior_ptid is referenced directly
instead of using the ptid passed as parameters.  I guess that it is
wrong, although I can't really test it in both cases.

gdb/ChangeLog:

	* bsd-uthread.c (bsd_uthread_thread_alive): Use ptid instead of
	inferior_ptid.
	* go32-nat.c (go32_thread_alive): Likewise.
This commit is contained in:
Simon Marchi 2017-02-23 10:36:20 -05:00
parent 60abdbedb2
commit 8eaf53202e
3 changed files with 8 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
gdb/ChangeLog
View file

@ -1,3 +1,9 @@
2017-02-23 Simon Marchi <simon.marchi@ericsson.com>
* bsd-uthread.c (bsd_uthread_thread_alive): Use ptid instead of
inferior_ptid.
* go32-nat.c (go32_thread_alive): Likewise.
2017-02-23 Yao Qi <yao.qi@linaro.org>
* varobj-iter.h (varobj_iter_delete): Call xfree instead of

2
gdb/bsd-uthread.c
View file

@ -401,7 +401,7 @@ bsd_uthread_thread_alive (struct target_ops *ops, ptid_t ptid)
{
enum bfd_endian byte_order = gdbarch_byte_order (target_gdbarch ());
struct target_ops *beneath = find_target_beneath (ops);
CORE_ADDR addr = ptid_get_tid (inferior_ptid);
CORE_ADDR addr = ptid_get_tid (ptid);
if (addr != 0)
{

2
gdb/go32-nat.c
View file

@ -938,7 +938,7 @@ go32_terminal_ours (struct target_ops *self)
static int
go32_thread_alive (struct target_ops *ops, ptid_t ptid)
{
return !ptid_equal (inferior_ptid, null_ptid);
return !ptid_equal (ptid, null_ptid);
}
static char *