commit 9dc998407042d1d110f1839be4a6b098ebd0e79e
author Parth Sane <parthsane@google.com> Tue Sep 17 18:07:22 2024 +0000
committer Parth Sane <parthsane@google.com> Tue Sep 17 18:07:22 2024 +0000
tree 499ebe1390d4d47929944897644ab7ad805e9e62
parent 69ec43c8a4007d96acf6d44a5484432b69ca884c

Fix Libbinder RemoveFromCacheOnServerDeath test

Get pid before killing the server

Test: atest binderCacheUnitTest
Bug: 333854840
Flag: Exempt Bugfix
Change-Id: I7d07c34e89867920f03545557eb51d4c307ec357

libs/binder/tests/binderCacheUnitTest.cpp

diff --git a/libs/binder/tests/binderCacheUnitTest.cpp b/libs/binder/tests/binderCacheUnitTest.cpp
index 92dab19..482d197 100644
--- a/libs/binder/tests/binderCacheUnitTest.cpp
+++ b/libs/binder/tests/binderCacheUnitTest.cpp
@@ -137,9 +137,9 @@
     ASSERT_EQ(binder1, result);
 
     // Kill the server, this should remove from cache.
-    foo.killServer(binder1);
     pid_t pid;
     ASSERT_EQ(OK, binder1->getDebugPid(&pid));
+    foo.killServer(binder1);
     system(("kill -9 " + std::to_string(pid)).c_str());
 
     sp<IBinder> binder2 = sp<BBinder>::make();