diff options
author | 2024-10-21 16:19:44 +0000 | |
---|---|---|
committer | 2024-10-21 16:19:44 +0000 | |
commit | a72b5858a085ae3b9433d9afd2d1a12e241be4aa (patch) | |
tree | 3e07e5bdea68ea2b8937fee884d666fc96c10a1e | |
parent | 9a9f68ff8ca7877b121fc5a1c3aeda5189f5ba06 (diff) | |
parent | 551db9811c434b12446a776d4c4d73d0fad1038c (diff) |
Merge "Document return values of ABinderRpc_Accessor_delegateAccessor" into main
-rw-r--r-- | libs/binder/ndk/binder_rpc.cpp | 4 | ||||
-rw-r--r-- | libs/binder/ndk/include_platform/android/binder_rpc.h | 5 |
2 files changed, 7 insertions, 2 deletions
diff --git a/libs/binder/ndk/binder_rpc.cpp b/libs/binder/ndk/binder_rpc.cpp index 07b8c40740..886eb4bdf8 100644 --- a/libs/binder/ndk/binder_rpc.cpp +++ b/libs/binder/ndk/binder_rpc.cpp @@ -255,7 +255,7 @@ ABinderRpc_Accessor* ABinderRpc_Accessor_new( "new variant was added to the ABinderRpc_ConnectionInfo and this needs to be " "updated."); } - return OK; + return STATUS_OK; }; sp<IBinder> accessorBinder = android::createAccessor(String16(instance), std::move(generate)); if (accessorBinder == nullptr) { @@ -321,7 +321,7 @@ binder_status_t ABinderRpc_Accessor_delegateAccessor(const char* instance, AIBin // This AIBinder needs a strong ref to pass ownership to the caller binder->incStrong(nullptr); *outDelegator = binder.get(); - return OK; + return STATUS_OK; } ABinderRpc_ConnectionInfo* ABinderRpc_ConnectionInfo_new(const sockaddr* addr, socklen_t len) { diff --git a/libs/binder/ndk/include_platform/android/binder_rpc.h b/libs/binder/ndk/include_platform/android/binder_rpc.h index 9fe5d788b4..66667d33bd 100644 --- a/libs/binder/ndk/include_platform/android/binder_rpc.h +++ b/libs/binder/ndk/include_platform/android/binder_rpc.h @@ -287,6 +287,11 @@ ABinderRpc_Accessor* _Nullable ABinderRpc_Accessor_fromBinder(const char* _Nonnu * this object with one strong ref count and is responsible for removing * that ref count with with AIBinder_decStrong when the caller wishes to * drop the reference. + * \return STATUS_OK on success. + * STATUS_UNEXPECTED_NULL if instance or binder arguments are null. + * STATUS_BAD_TYPE if the binder is not an IAccessor. + * STATUS_NAME_NOT_FOUND if the binder is an IAccessor, but not + * associated with the provided instance name. */ binder_status_t ABinderRpc_Accessor_delegateAccessor(const char* _Nonnull instance, AIBinder* _Nonnull binder, |