Merge "[getService] Only set binder when getService2() succeeds" into main am: 33a6e89

Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/3441659

Change-Id: I5bf8e49c17d58548e72d167914b3e3c638a03098
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

Author: Treehugger Robot

libs/binder/BackendUnifiedServiceManager.cpp

@@ -215,7 +215,9 @@ Status BackendUnifiedServiceManager::getService(const ::std::string& name,
                                                 sp<IBinder>* _aidl_return) {
     os::Service service;
     Status status = getService2(name, &service);
-    *_aidl_return = service.get<os::Service::Tag::serviceWithMetadata>().service;
+    if (status.isOk()) {
+        *_aidl_return = service.get<os::Service::Tag::serviceWithMetadata>().service;
+    }
     return status;
 }