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

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

commit 3dd36d1eb4666abf6a6d63829eec05819a51e229
Author: PengZheng <[email protected]>
AuthorDate: Mon Apr 10 17:53:56 2023 +0800

    Fix Coverity 218467 Logically dead code.
---
 libs/framework/src/framework.c | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/libs/framework/src/framework.c b/libs/framework/src/framework.c
index 97f08459..8114d7cc 100644
--- a/libs/framework/src/framework.c
+++ b/libs/framework/src/framework.c
@@ -1943,12 +1943,8 @@ celix_status_t 
celix_framework_uninstallBundleEntry(celix_framework_t* framework
         celix_bundle_t *bnd = NULL;
         long bndId = -1L;
 
-        if (removedEntry == NULL) {
-            status = CELIX_ILLEGAL_ARGUMENT;
-        } else {
-            bnd = removedEntry->bnd;
-            bndId = removedEntry->bndId;
-        }
+        bnd = removedEntry->bnd;
+        bndId = removedEntry->bndId;
 
         if (status == CELIX_SUCCESS) {
             bundle_archive_t *archive = NULL;

Reply via email to