commit af4aba557ba4a7f9fe22a23ad6e26476737f824a
author Martijn Coenen <maco@google.com> Thu Apr 27 09:41:13 2017 -0700
committer Martijn Coenen <maco@google.com> Thu Apr 27 09:41:13 2017 -0700
tree 9cafa5561e07d9166def8ea9df9faa11bdfeebac
parent 81e25c13192462a60eda925413302d0f8c6a936e

Make registerPassthroughClient() synchronous.

And remove the PID argument.

Bug: 37730021
Test: lshal shows passthrough clients again
Change-Id: I0bee20ae5cde36dbc86662654d0ac9e9489bf0a2

transport/ServiceManagement.cpp

diff --git a/transport/ServiceManagement.cpp b/transport/ServiceManagement.cpp
index 9c692da..617a858 100644
--- a/transport/ServiceManagement.cpp
+++ b/transport/ServiceManagement.cpp
@@ -201,7 +201,7 @@
                      << ": null binderized manager.";
         return;
     }
-    auto ret = binderizedManager->registerPassthroughClient(interfaceName, instanceName, getpid());
+    auto ret = binderizedManager->registerPassthroughClient(interfaceName, instanceName);
     if (!ret.isOk()) {
         LOG(WARNING) << "Could not registerReference for "
                      << interfaceName << "/" << instanceName
@@ -369,7 +369,7 @@
         return Void();
     }
 
-    Return<void> registerPassthroughClient(const hidl_string &, const hidl_string &, int32_t) override {
+    Return<void> registerPassthroughClient(const hidl_string &, const hidl_string &) override {
         // This makes no sense.
         LOG(FATAL) << "Cannot call registerPassthroughClient on passthrough service manager. "
                    << "Call it on defaultServiceManager() instead.";