diff options
author | 2025-01-09 09:01:52 -0800 | |
---|---|---|
committer | 2025-01-09 09:01:52 -0800 | |
commit | 33a6e89aa8ebde305ea3bc408d3b5b96756cacd6 (patch) | |
tree | 4f022d279b7e064bcfe5ebd1adcd1c68bdbba382 /libs/binder/BackendUnifiedServiceManager.cpp | |
parent | a11599667a5420cb53866e2e75bbdc19c666609a (diff) | |
parent | 0aa07a1179c27192a99daf76579652413f840018 (diff) |
Merge "[getService] Only set binder when getService2() succeeds" into main
Diffstat (limited to 'libs/binder/BackendUnifiedServiceManager.cpp')
-rw-r--r-- | libs/binder/BackendUnifiedServiceManager.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/libs/binder/BackendUnifiedServiceManager.cpp b/libs/binder/BackendUnifiedServiceManager.cpp index 4220cdd0e8..7c0319aead 100644 --- a/libs/binder/BackendUnifiedServiceManager.cpp +++ b/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; } |