Zbigniew Jędrzejewski-Szmek 126222
From 106c98f6b08d3b9254b419b09bc4ceed1963d7aa Mon Sep 17 00:00:00 2001
Zbigniew Jędrzejewski-Szmek 126222
From: =?UTF-8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= <zbyszek@in.waw.pl>
Zbigniew Jędrzejewski-Szmek 126222
Date: Tue, 14 Jan 2014 17:31:00 -0500
Zbigniew Jędrzejewski-Szmek 126222
Subject: [PATCH] Revert "systemd: add a start job for all units specified with
Zbigniew Jędrzejewski-Szmek 126222
 SYSTEMD_WANTS="
Zbigniew Jędrzejewski-Szmek 126222
Zbigniew Jędrzejewski-Szmek 126222
This reverts commit 043a559ff3732439fc61872a6320ee0a05dd088f.
Zbigniew Jędrzejewski-Szmek 126222
Zbigniew Jędrzejewski-Szmek 126222
This was a mistake, and was reverted upstream.
Zbigniew Jędrzejewski-Szmek 126222
---
Zbigniew Jędrzejewski-Szmek 126222
 src/core/device.c | 14 +++-----------
Zbigniew Jędrzejewski-Szmek 126222
 1 file changed, 3 insertions(+), 11 deletions(-)
Zbigniew Jędrzejewski-Szmek 126222
Zbigniew Jędrzejewski-Szmek 126222
diff --git a/src/core/device.c b/src/core/device.c
Zbigniew Jędrzejewski-Szmek 126222
index e02c207..f79c206 100644
Zbigniew Jędrzejewski-Szmek 126222
--- a/src/core/device.c
Zbigniew Jędrzejewski-Szmek 126222
+++ b/src/core/device.c
Zbigniew Jędrzejewski-Szmek 126222
@@ -282,8 +282,7 @@ static int device_update_unit(Manager *m, struct udev_device *dev, const char *p
Zbigniew Jędrzejewski-Szmek 126222
                         size_t l;
Zbigniew Jędrzejewski-Szmek 126222
 
Zbigniew Jędrzejewski-Szmek 126222
                         FOREACH_WORD_QUOTED(w, l, wants, state) {
Zbigniew Jędrzejewski-Szmek 126222
-                                _cleanup_free_ char *e, *n = NULL;
Zbigniew Jędrzejewski-Szmek 126222
-                                Unit *other;
Zbigniew Jędrzejewski-Szmek 126222
+                                char *e, *n;
Zbigniew Jędrzejewski-Szmek 126222
 
Zbigniew Jędrzejewski-Szmek 126222
                                 e = strndup(w, l);
Zbigniew Jędrzejewski-Szmek 126222
                                 if (!e) {
Zbigniew Jędrzejewski-Szmek 126222
@@ -295,19 +294,12 @@ static int device_update_unit(Manager *m, struct udev_device *dev, const char *p
Zbigniew Jędrzejewski-Szmek 126222
                                         r = -ENOMEM;
Zbigniew Jędrzejewski-Szmek 126222
                                         goto fail;
Zbigniew Jędrzejewski-Szmek 126222
                                 }
Zbigniew Jędrzejewski-Szmek 126222
+                                free(e);
Zbigniew Jędrzejewski-Szmek 126222
 
Zbigniew Jędrzejewski-Szmek 126222
                                 r = unit_add_dependency_by_name(u, UNIT_WANTS, n, NULL, true);
Zbigniew Jędrzejewski-Szmek 126222
+                                free(n);
Zbigniew Jędrzejewski-Szmek 126222
                                 if (r < 0)
Zbigniew Jędrzejewski-Szmek 126222
                                         goto fail;
Zbigniew Jędrzejewski-Szmek 126222
-
Zbigniew Jędrzejewski-Szmek 126222
-                                other = manager_get_unit(u->manager, n);
Zbigniew Jędrzejewski-Szmek 126222
-                                if (!other || !unit_can_start(other))
Zbigniew Jędrzejewski-Szmek 126222
-                                        continue;
Zbigniew Jędrzejewski-Szmek 126222
-
Zbigniew Jędrzejewski-Szmek 126222
-                                r = manager_add_job(u->manager, JOB_START, other, JOB_REPLACE, true, NULL, NULL);
Zbigniew Jędrzejewski-Szmek 126222
-                                if (r < 0)
Zbigniew Jędrzejewski-Szmek 126222
-                                        log_warning("Failed to add job %s/%s, ignoring: %s.",
Zbigniew Jędrzejewski-Szmek 126222
-                                                    other->id, job_type_to_string(JOB_START), strerror(-r));
Zbigniew Jędrzejewski-Szmek 126222
                         }
Zbigniew Jędrzejewski-Szmek 126222
                 }
Zbigniew Jędrzejewski-Szmek 126222
         }