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/03 13:35:51 UTC
celix git commit: CELIX-269: Removed double free from calculator
shell for remote services test
Repository: celix
Updated Branches:
refs/heads/release/celix-2.0.0 9441ac2de -> a0f2217a0
CELIX-269: Removed double free from calculator shell for remote services test
Project: http://git-wip-us.apache.org/repos/asf/celix/repo
Commit: http://git-wip-us.apache.org/repos/asf/celix/commit/a0f2217a
Tree: http://git-wip-us.apache.org/repos/asf/celix/tree/a0f2217a
Diff: http://git-wip-us.apache.org/repos/asf/celix/diff/a0f2217a
Branch: refs/heads/release/celix-2.0.0
Commit: a0f2217a08be60737b0c2a63e7899159d07dcc1d
Parents: 9441ac2
Author: Pepijn Noltes <pe...@gmail.com>
Authored: Tue Nov 3 13:34:58 2015 +0100
Committer: Pepijn Noltes <pe...@gmail.com>
Committed: Tue Nov 3 13:34:58 2015 +0100
----------------------------------------------------------------------
.../private/src/calculator_shell_activator.c | 8 +-------
1 file changed, 1 insertion(+), 7 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/celix/blob/a0f2217a/remote_services/examples/calculator_shell/private/src/calculator_shell_activator.c
----------------------------------------------------------------------
diff --git a/remote_services/examples/calculator_shell/private/src/calculator_shell_activator.c b/remote_services/examples/calculator_shell/private/src/calculator_shell_activator.c
index ddbe842..29fb49e 100644
--- a/remote_services/examples/calculator_shell/private/src/calculator_shell_activator.c
+++ b/remote_services/examples/calculator_shell/private/src/calculator_shell_activator.c
@@ -114,13 +114,7 @@ celix_status_t bundleActivator_stop(void * userData, bundle_context_pt context)
free(activator->addCmdSrv);
free(activator->subCmdSrv);
free(activator->sqrtCmdSrv);
-
- if (status == CELIX_SUCCESS) {
- free(activator->addCmdSrv);
- free(activator->sqrtCmdSrv);
- free(activator->subCmdSrv);
- }
-
+
return status;
}