commit 59f60fd466ec36b9cce844bcbce348790eb9a7e5
author Elie Kheirallah <khei@google.com> Thu Jun 09 23:59:04 2022 +0000
committer Elie Kheirallah <khei@google.com> Fri Jun 10 19:39:02 2022 +0000
tree 4b1678d85611696dcccd7e5a5aa6bbf79c2658c6
parent c56d0ecb4f698195b09cf62799028abf6a014015

binderLibTest HangingServices and ThreadPoolAvailableThreads fix

Changed reply to local reply for threads.

Bug: 235494189
Test: adb shell /data/nativetest64/binderLibTest/binderLibTest // over 50 runs
Change-Id: Ie1ceeebdd3e77a6f678bb16c8cc0585917ac02ee

libs/binder/tests/binderLibTest.cpp

diff --git a/libs/binder/tests/binderLibTest.cpp b/libs/binder/tests/binderLibTest.cpp
index 3e90726..cf3e6ca 100644
--- a/libs/binder/tests/binderLibTest.cpp
+++ b/libs/binder/tests/binderLibTest.cpp
@@ -1264,7 +1264,9 @@
     std::vector<std::thread> ts;
     for (size_t i = 0; i < kKernelThreads - 1; i++) {
         ts.push_back(std::thread([&] {
-            EXPECT_THAT(server->transact(BINDER_LIB_TEST_LOCK_UNLOCK, data, &reply), NO_ERROR);
+            Parcel local_reply;
+            EXPECT_THAT(server->transact(BINDER_LIB_TEST_LOCK_UNLOCK, data, &local_reply),
+                        NO_ERROR);
         }));
     }
 
@@ -1302,7 +1304,9 @@
     size_t epochMsBefore = epochMillis();
     for (size_t i = 0; i < kKernelThreads + 1; i++) {
         ts.push_back(std::thread([&] {
-            EXPECT_THAT(server->transact(BINDER_LIB_TEST_LOCK_UNLOCK, data, &reply), NO_ERROR);
+            Parcel local_reply;
+            EXPECT_THAT(server->transact(BINDER_LIB_TEST_LOCK_UNLOCK, data, &local_reply),
+                        NO_ERROR);
         }));
     }