diff --git a/core/jni/android_os_HwBinder.cpp b/core/jni/android_os_HwBinder.cpp index 816d5df1c710a..1a33d9111fdb8 100644 --- a/core/jni/android_os_HwBinder.cpp +++ b/core/jni/android_os_HwBinder.cpp @@ -24,8 +24,9 @@ #include "android_os_HwRemoteBinder.h" #include +#include #include -#include +#include #include #include #include @@ -220,27 +221,31 @@ static void JHwBinder_native_registerService( return; // XXX exception already pending? } - const hardware::hidl_version kVersion = - hardware::make_hidl_version(versionMajor, versionMinor); + using android::hidl::manager::V1_0::IServiceManager; + + const IServiceManager::Version kVersion { + .major = static_cast(versionMajor), + .minor = static_cast(versionMinor), + }; sp binder = JHwBinder::GetNativeContext(env, thiz); - status_t err = hardware::defaultServiceManager()->addService( - String16( - reinterpret_cast(serviceName), - env->GetStringLength(serviceNameObj)), + bool ok = hardware::defaultServiceManager()->add( + String8(String16( + reinterpret_cast(serviceName), + env->GetStringLength(serviceNameObj))).string(), binder, kVersion); env->ReleaseStringCritical(serviceNameObj, serviceName); serviceName = NULL; - if (err == OK) { + if (ok) { LOG(INFO) << "Starting thread pool."; ::android::hardware::ProcessState::self()->startThreadPool(); } - signalExceptionForError(env, err); + signalExceptionForError(env, (ok ? OK : UNKNOWN_ERROR)); } static jobject JHwBinder_native_getService( @@ -268,8 +273,12 @@ static jobject JHwBinder_native_getService( return NULL; // XXX exception already pending? } - const hardware::hidl_version kVersion = - hardware::make_hidl_version(versionMajor, versionMinor); + using android::hidl::manager::V1_0::IServiceManager; + + const IServiceManager::Version kVersion { + .major = static_cast(versionMajor), + .minor = static_cast(versionMinor), + }; LOG(INFO) << "looking for service '" << String8(String16( @@ -277,12 +286,15 @@ static jobject JHwBinder_native_getService( env->GetStringLength(serviceNameObj))).string() << "'"; - sp service = - hardware::defaultServiceManager()->getService( - String16( - reinterpret_cast(serviceName), - env->GetStringLength(serviceNameObj)), - kVersion); + sp service; + hardware::defaultServiceManager()->get( + String8(String16( + reinterpret_cast(serviceName), + env->GetStringLength(serviceNameObj))).string(), + kVersion, + [&service](sp out) { + service = out; + }); env->ReleaseStringCritical(serviceNameObj, serviceName); serviceName = NULL; diff --git a/core/jni/android_os_HwRemoteBinder.cpp b/core/jni/android_os_HwRemoteBinder.cpp index 3023ba87d24b0..1d5d6d59639a0 100644 --- a/core/jni/android_os_HwRemoteBinder.cpp +++ b/core/jni/android_os_HwRemoteBinder.cpp @@ -24,7 +24,6 @@ #include #include -#include #include #include