You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@celix.apache.org by pn...@apache.org on 2015/11/16 21:17:34 UTC
[20/21] celix git commit: CELIX-272: Fix mem leak in rsa dfi
CELIX-272: Fix mem leak in rsa dfi
Project: http://git-wip-us.apache.org/repos/asf/celix/repo
Commit: http://git-wip-us.apache.org/repos/asf/celix/commit/d0dbdb38
Tree: http://git-wip-us.apache.org/repos/asf/celix/tree/d0dbdb38
Diff: http://git-wip-us.apache.org/repos/asf/celix/diff/d0dbdb38
Branch: refs/heads/feature/CELIX-272_synchronization_service_registry
Commit: d0dbdb38d30f5092fb06d585f206755712471de4
Parents: 0a177d9
Author: Pepijn Noltes <pe...@gmail.com>
Authored: Mon Nov 16 21:16:10 2015 +0100
Committer: Pepijn Noltes <pe...@gmail.com>
Committed: Mon Nov 16 21:16:10 2015 +0100
----------------------------------------------------------------------
.../rsa/private/src/remote_service_admin_dfi.c | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/celix/blob/d0dbdb38/remote_services/remote_service_admin_dfi/rsa/private/src/remote_service_admin_dfi.c
----------------------------------------------------------------------
diff --git a/remote_services/remote_service_admin_dfi/rsa/private/src/remote_service_admin_dfi.c b/remote_services/remote_service_admin_dfi/rsa/private/src/remote_service_admin_dfi.c
index fc0303b..87a494a 100644
--- a/remote_services/remote_service_admin_dfi/rsa/private/src/remote_service_admin_dfi.c
+++ b/remote_services/remote_service_admin_dfi/rsa/private/src/remote_service_admin_dfi.c
@@ -388,10 +388,7 @@ celix_status_t remoteServiceAdmin_exportService(remote_service_admin_pt admin, c
bundleContext_ungetServiceReference(admin->context, arrayList_get(references, i));
}
}
-
- if(references != NULL){
- arrayList_destroy(references);
- }
+ arrayList_destroy(references);
if (reference == NULL) {
logHelper_log(admin->loghelper, OSGI_LOGSERVICE_ERROR, "ERROR: expected a reference for service id %s.", serviceId);