CELIX-418: Fixed memleak in celix_launcher
Project: http://git-wip-us.apache.org/repos/asf/celix/repo Commit: http://git-wip-us.apache.org/repos/asf/celix/commit/9ff00c30 Tree: http://git-wip-us.apache.org/repos/asf/celix/tree/9ff00c30 Diff: http://git-wip-us.apache.org/repos/asf/celix/diff/9ff00c30 Branch: refs/heads/master Commit: 9ff00c30602e7a069f4a4af7ac9763e56efbd330 Parents: 5c9e52e Author: Roy Lenferink <[email protected]> Authored: Wed Jan 3 08:37:47 2018 +0100 Committer: Roy Lenferink <[email protected]> Committed: Wed Jan 3 08:37:47 2018 +0100 ---------------------------------------------------------------------- framework/private/src/celix_launcher.c | 5 +++++ 1 file changed, 5 insertions(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/celix/blob/9ff00c30/framework/private/src/celix_launcher.c ---------------------------------------------------------------------- diff --git a/framework/private/src/celix_launcher.c b/framework/private/src/celix_launcher.c index 4732b15..16410bc 100644 --- a/framework/private/src/celix_launcher.c +++ b/framework/private/src/celix_launcher.c @@ -197,6 +197,11 @@ static int celixLauncher_launchWithStreamAndProps(FILE *stream, framework_pt *fr value = (const char *) hashMapEntry_getValue(entry); } } + + // normally, the framework_destroy will clean up the properties_pt + // since there are 2 properties_pt available (runtimeConfig and packedConfig) + // the packedConfig must be destroyed + properties_destroy(packedConfig); } status = celixLauncher_launchWithProperties(runtimeConfig, framework);
