commit 530552ac735c59a3ae44b16f751df265e820dad2
author wangguibo <wangguibo@xiaomi.com> Sun Jun 06 09:26:07 2021 +0800
committer Hans Boehm <hboehm@google.com> Sun Jun 06 18:35:48 2021 +0000
tree 2ca5654c5152df395af60811ceb0dc891e1137b7
parent bbc4dc3f98bdc2f984371b39ea5a96d85a3ab2a1

Fix DumpUnattachedThreads logic for CTS.

The thread should not be in thread list when dumping from
DumpUnattachedThread.

Bug:140968481
Bug:188918448
Bug:188918454
Test:CtsWindowManagerDeviceTestCases android.server.wm.AnrTests

Signed-off-by: wangguibo <wangguibo@xiaomi.com>
Change-Id: Icc4c7907c5386280c04d8e56496361e32cbac350

runtime/thread_list.cc

diff --git a/runtime/thread_list.cc b/runtime/thread_list.cc
index efc354c..84b7384 100644
--- a/runtime/thread_list.cc
+++ b/runtime/thread_list.cc
@@ -175,7 +175,7 @@
         MutexLock mu(self, *Locks::thread_list_lock_);
         thread = FindThreadByTid(tid);
       }
-      if (thread != nullptr) {
+      if (thread == nullptr) {
         DumpUnattachedThread(os, tid, dump_native_stack);
       }
     }