cgroup: units that aren't loaded properly should not result in cgroup controllers being pulled in

This shouldn't make much difference in real life, but is a bit cleaner.
2018-11-23 12:24:37 +01:00
parent 0adf88b68c
commit 442ce7759c
1 changed files with 5 additions and 1 deletions
@@ -1270,7 +1270,11 @@ static CGroupMask unit_get_bpf_mask(Unit *u) {
 CGroupMask unit_get_own_mask(Unit *u) {
        CGroupContext *c;

-        /* Returns the mask of controllers the unit needs for itself */
+        /* Returns the mask of controllers the unit needs for itself. If a unit is not properly loaded, return an empty
+         * mask, as we shouldn't reflect it in the cgroup hierarchy then. */
+
+        if (u->load_state != UNIT_LOADED)
+                return 0;

        c = unit_get_cgroup_context(u);
        if (!c)