[celix] 01/01: Fix double-dlclose and remove dead code.

[pengzheng]

This is an automated email from the ASF dual-hosted git repository.

pengzheng pushed a commit to branch hotfix/double-dlclose
in repository https://gitbox.apache.org/repos/asf/celix.git

commit d37e8570a6c39428cd03f4dabe03d4ba0e40540b
Author: PengZheng <howto...@gmail.com>
AuthorDate: Thu Apr 27 22:21:36 2023 +0800

    Fix double-dlclose and remove dead code.
---
 libs/framework/src/module.c | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/libs/framework/src/module.c b/libs/framework/src/module.c
index 2267622c..afc559b1 100644
--- a/libs/framework/src/module.c
+++ b/libs/framework/src/module.c
@@ -251,12 +251,12 @@ celix_status_t module_getGroup(module_pt module, const 
char **symbolicName) {
     return status;
 }
 
-celix_status_t module_getDescription(module_pt module, const char 
**symbolicName) {
+celix_status_t module_getDescription(module_pt module, const char 
**descriptoin) {
     celix_status_t status = CELIX_SUCCESS;
     if (module == NULL) {
         status = CELIX_ILLEGAL_ARGUMENT;
     } else {
-        *symbolicName = module->description;
+        *descriptoin = module->description;
     }
     return status;
 }
@@ -454,10 +454,7 @@ static celix_status_t 
celix_module_loadLibrariesInManifestEntry(celix_module_t*
 
         if ( (status == CELIX_SUCCESS) && (activator != NULL) && 
(strcmp(trimmedLib, activator) == 0) ) {
             *activatorHandle = handle;
-        } else if ((status != CELIX_SUCCESS) && (handle != NULL)) {
-            celix_libloader_close(fwCtx, handle);
         }
-
         token = strtok_r(NULL, ",", &last);
     }
 
@@ -501,8 +498,6 @@ celix_status_t celix_module_loadLibraries(celix_module_t* 
module) {
             celixThreadMutex_lock(&module->handlesLock);
             module->bundleActivatorHandle = activatorHandle;
             celixThreadMutex_unlock(&module->handlesLock);
-        } else if (activatorHandle != NULL) {
-            celix_libloader_close(fwCtx, activatorHandle);
         }
     }