[systemd-commits] src/core

Lennart Poettering lennart at kemper.freedesktop.org
Fri Apr 24 08:33:05 PDT 2015


 src/core/unit.c |   10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

New commits:
commit 47bc12e1ba35d38edda737dae232088d6d3ae688
Author: Lennart Poettering <lennart at poettering.net>
Date:   Fri Apr 24 17:28:06 2015 +0200

    unit: don't add automatic dependencies on device units if they aren't supported
    
    http://lists.freedesktop.org/archives/systemd-devel/2015-April/031187.html

diff --git a/src/core/unit.c b/src/core/unit.c
index 2b356e2..730492e 100644
--- a/src/core/unit.c
+++ b/src/core/unit.c
@@ -2846,14 +2846,18 @@ int unit_add_node_link(Unit *u, const char *what, bool wants) {
 
         assert(u);
 
-        if (!what)
-                return 0;
-
         /* Adds in links to the device node that this unit is based on */
+        if (isempty(what))
+                return 0;
 
         if (!is_device_path(what))
                 return 0;
 
+        /* When device units aren't supported (such as in a
+         * container), don't create dependencies on them. */
+        if (unit_vtable[UNIT_DEVICE]->supported && !unit_vtable[UNIT_DEVICE]->supported(u->manager))
+                return 0;
+
         e = unit_name_from_path(what, ".device");
         if (!e)
                 return -ENOMEM;



More information about the systemd-commits mailing list