Zbigniew Jędrzejewski-Szmek 708deb
From e63ff941eaa012ffd003e3791438c73b8a26227d Mon Sep 17 00:00:00 2001
Zbigniew Jędrzejewski-Szmek 708deb
From: Tom Gundersen <teg@jklm.no>
Zbigniew Jędrzejewski-Szmek 708deb
Date: Tue, 30 Sep 2014 11:34:01 +0200
Zbigniew Jędrzejewski-Szmek 708deb
Subject: [PATCH] core: execute - don't leak strv
Zbigniew Jędrzejewski-Szmek 708deb
Zbigniew Jędrzejewski-Szmek 708deb
---
Zbigniew Jędrzejewski-Szmek 708deb
 src/core/execute.c | 6 ++----
Zbigniew Jędrzejewski-Szmek 708deb
 1 file changed, 2 insertions(+), 4 deletions(-)
Zbigniew Jędrzejewski-Szmek 708deb
Zbigniew Jędrzejewski-Szmek 708deb
diff --git a/src/core/execute.c b/src/core/execute.c
Zbigniew Jędrzejewski-Szmek 708deb
index 07ec7a28d6..8b9bb27137 100644
Zbigniew Jędrzejewski-Szmek 708deb
--- a/src/core/execute.c
Zbigniew Jędrzejewski-Szmek 708deb
+++ b/src/core/execute.c
Zbigniew Jędrzejewski-Szmek 708deb
@@ -2567,8 +2567,8 @@ int exec_command_set(ExecCommand *c, const char *path, ...) {
Zbigniew Jędrzejewski-Szmek 708deb
 }
Zbigniew Jędrzejewski-Szmek 708deb
 
Zbigniew Jędrzejewski-Szmek 708deb
 int exec_command_append(ExecCommand *c, const char *path, ...) {
Zbigniew Jędrzejewski-Szmek 708deb
+        _cleanup_strv_free_ char **l = NULL;
Zbigniew Jędrzejewski-Szmek 708deb
         va_list ap;
Zbigniew Jędrzejewski-Szmek 708deb
-        char **l;
Zbigniew Jędrzejewski-Szmek 708deb
         int r;
Zbigniew Jędrzejewski-Szmek 708deb
 
Zbigniew Jędrzejewski-Szmek 708deb
         assert(c);
Zbigniew Jędrzejewski-Szmek 708deb
@@ -2582,10 +2582,8 @@ int exec_command_append(ExecCommand *c, const char *path, ...) {
Zbigniew Jędrzejewski-Szmek 708deb
                 return -ENOMEM;
Zbigniew Jędrzejewski-Szmek 708deb
 
Zbigniew Jędrzejewski-Szmek 708deb
         r = strv_extend_strv(&c->argv, l);
Zbigniew Jędrzejewski-Szmek 708deb
-        if (r < 0) {
Zbigniew Jędrzejewski-Szmek 708deb
-                strv_free(l);
Zbigniew Jędrzejewski-Szmek 708deb
+        if (r < 0)
Zbigniew Jędrzejewski-Szmek 708deb
                 return r;
Zbigniew Jędrzejewski-Szmek 708deb
-        }
Zbigniew Jędrzejewski-Szmek 708deb
 
Zbigniew Jędrzejewski-Szmek 708deb
         return 0;
Zbigniew Jędrzejewski-Szmek 708deb
 }