From 69178dc45cccabbca20aaabb54e9e5e841dd8c4f Mon Sep 17 00:00:00 2001 From: Steven Moreland Date: Tue, 9 Feb 2021 00:17:29 +0000 Subject: libbinder_ndk's addService ((warn_unused_result)) Bug: 179713096 Test: mma Change-Id: I52daf67faa7e9a519115c3910b186f9a3356955e --- libs/binder/ndk/include_platform/android/binder_manager.h | 3 ++- libs/binder/ndk/tests/binderVendorDoubleLoadTest.cpp | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/libs/binder/ndk/include_platform/android/binder_manager.h b/libs/binder/ndk/include_platform/android/binder_manager.h index 2784aa823d..e13d4ccecd 100644 --- a/libs/binder/ndk/include_platform/android/binder_manager.h +++ b/libs/binder/ndk/include_platform/android/binder_manager.h @@ -31,7 +31,8 @@ __BEGIN_DECLS * * \return EX_NONE on success. */ -binder_exception_t AServiceManager_addService(AIBinder* binder, const char* instance); +__attribute__((warn_unused_result)) binder_exception_t AServiceManager_addService( + AIBinder* binder, const char* instance); /** * Gets a binder object with this specific instance name. Will return nullptr immediately if the diff --git a/libs/binder/ndk/tests/binderVendorDoubleLoadTest.cpp b/libs/binder/ndk/tests/binderVendorDoubleLoadTest.cpp index ad78e319a2..f3cd21822d 100644 --- a/libs/binder/ndk/tests/binderVendorDoubleLoadTest.cpp +++ b/libs/binder/ndk/tests/binderVendorDoubleLoadTest.cpp @@ -163,7 +163,8 @@ int main(int argc, char** argv) { // LOCAL SERVERS std::shared_ptr ndkServer = SharedRefBase::make(); - AServiceManager_addService(ndkServer->asBinder().get(), kLocalNdkServerName.c_str()); + CHECK(STATUS_OK == + AServiceManager_addService(ndkServer->asBinder().get(), kLocalNdkServerName.c_str())); return RUN_ALL_TESTS(); } -- cgit v1.2.3-59-g8ed1b