From: Cristian Rodríguez Date: Thu, 18 Sep 2014 00:56:25 +0000 (-0300) Subject: systemctl: fix resource leak CID #1237747 X-Git-Tag: v217~532 X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?p=elogind.git;a=commitdiff_plain;h=48a2900c6612052149a1d0dd88aeacb99b49ce4d systemctl: fix resource leak CID #1237747 ..by simply moving the declaration of "unit" into the STRV_FOREACH loop as suggested by Andreas. --- diff --git a/src/systemctl/systemctl.c b/src/systemctl/systemctl.c index 88be871f3..901212852 100644 --- a/src/systemctl/systemctl.c +++ b/src/systemctl/systemctl.c @@ -4449,7 +4449,6 @@ static int show(sd_bus *bus, char **args) { } static int cat(sd_bus *bus, char **args) { - _cleanup_free_ char *unit = NULL; _cleanup_strv_free_ char **names = NULL; char **name; bool first = true; @@ -4468,6 +4467,8 @@ static int cat(sd_bus *bus, char **args) { _cleanup_bus_error_free_ sd_bus_error error = SD_BUS_ERROR_NULL; _cleanup_strv_free_ char **dropin_paths = NULL; _cleanup_free_ char *fragment_path = NULL; + _cleanup_free_ char *unit = NULL; + char **path; unit = unit_dbus_path_from_name(*name);