ryantimwilson / rpms / systemd

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