summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLennart Poettering <lennart@poettering.net>2015-01-14 22:31:03 +0100
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>2015-01-16 09:40:08 -0500
commit7bc3b3007e7ebd04f88df6bc2c380a9f6d075190 (patch)
tree3e18446ca421d1822009e6c09fe50aeafbe5e4a0
parent1b6db97fa2f5d537f3d2ee894b66df9751918247 (diff)
downloadsystemd-7bc3b3007e7ebd04f88df6bc2c380a9f6d075190.tar.gz
sysv-generator: fix memory leak on failure
This fixes a memory leak introduced by 1ed0c19f81fd13cdf283c6def0168ce122a853a9 (cherry picked from commit b3fae863ef548add2d01c3956ce7720f4eeeca7e)
-rw-r--r--src/sysv-generator/sysv-generator.c14
1 files changed, 9 insertions, 5 deletions
diff --git a/src/sysv-generator/sysv-generator.c b/src/sysv-generator/sysv-generator.c
index 101dfddc38..6f87f63b72 100644
--- a/src/sysv-generator/sysv-generator.c
+++ b/src/sysv-generator/sysv-generator.c
@@ -760,13 +760,11 @@ static int enumerate_sysv(LookupPaths lp, Hashmap *all_services) {
service->name = name;
service->path = fpath;
- r = load_sysv(service);
- if (r < 0)
- continue;
-
r = hashmap_put(all_services, service->name, service);
- if (r < 0)
+ if (r < 0) {
+ free(service);
return log_oom();
+ }
name = fpath = NULL;
}
@@ -948,6 +946,12 @@ int main(int argc, char *argv[]) {
}
HASHMAP_FOREACH(service, all_services, j) {
+ q = load_sysv(service);
+ if (q < 0)
+ continue;
+ }
+
+ HASHMAP_FOREACH(service, all_services, j) {
q = fix_order(service, all_services);
if (q < 0)
continue;