diff --git a/libnativebridge/include/nativebridge/native_bridge.h b/libnativebridge/include/nativebridge/native_bridge.h index 9bfc93523..28f192748 100644 --- a/libnativebridge/include/nativebridge/native_bridge.h +++ b/libnativebridge/include/nativebridge/native_bridge.h @@ -99,7 +99,7 @@ bool NativeBridgeError(); bool NativeBridgeNameAcceptable(const char* native_bridge_library_filename); // Decrements the reference count on the dynamic library handler. If the reference count drops -// to zero then the dynamic library is unloaded. +// to zero then the dynamic library is unloaded. Returns 0 on success and non-zero on error. int NativeBridgeUnloadLibrary(void* handle); // Get last error message of native bridge when fail to load library or search symbol. diff --git a/libnativeloader/include/nativeloader/native_loader.h b/libnativeloader/include/nativeloader/native_loader.h index 19a17832e..c1d9d2a92 100644 --- a/libnativeloader/include/nativeloader/native_loader.h +++ b/libnativeloader/include/nativeloader/native_loader.h @@ -47,8 +47,9 @@ void* OpenNativeLibrary(JNIEnv* env, bool* needs_native_bridge, std::string* error_msg); -__attribute__((visibility("default"))) -bool CloseNativeLibrary(void* handle, const bool needs_native_bridge); +__attribute__((visibility("default"))) bool CloseNativeLibrary(void* handle, + const bool needs_native_bridge, + std::string* error_msg); #if defined(__ANDROID__) // Look up linker namespace by class_loader. Returns nullptr if diff --git a/libnativeloader/native_loader.cpp b/libnativeloader/native_loader.cpp index 67c1c103b..b3e2b97fe 100644 --- a/libnativeloader/native_loader.cpp +++ b/libnativeloader/native_loader.cpp @@ -710,9 +710,21 @@ void* OpenNativeLibrary(JNIEnv* env, #endif } -bool CloseNativeLibrary(void* handle, const bool needs_native_bridge) { - return needs_native_bridge ? NativeBridgeUnloadLibrary(handle) : - dlclose(handle); +bool CloseNativeLibrary(void* handle, const bool needs_native_bridge, std::string* error_msg) { + bool success; + if (needs_native_bridge) { + success = (NativeBridgeUnloadLibrary(handle) == 0); + if (!success) { + *error_msg = NativeBridgeGetError(); + } + } else { + success = (dlclose(handle) == 0); + if (!success) { + *error_msg = dlerror(); + } + } + + return success; } #if defined(__ANDROID__)