From ac380e43a4f850ea212247639ba5ab80c58d73f0 Mon Sep 17 00:00:00 2001 From: msizanoen1 Date: Jun 01 2023 14:37:09 +0000 Subject: core: Do not check child freezability when thawing slice We want thawing operations to still succeed even in the presence of an unfreezable unit type (e.g. mount) appearing under a slice after the slice was frozen. The appearance of such units should never cause the slice thawing operation to fail to prevent potential future repeats of https://github.com/systemd/systemd/issues/25356. (cherry picked from commit b458659a1619337fa83353b5f41bae35d7ad4c69) --- diff --git a/src/core/slice.c b/src/core/slice.c index 4824a30..a1c7b45 100644 --- a/src/core/slice.c +++ b/src/core/slice.c @@ -375,7 +375,7 @@ static int slice_freezer_action(Unit *s, FreezerAction action) { assert(s); assert(IN_SET(action, FREEZER_FREEZE, FREEZER_THAW)); - if (!slice_freezer_action_supported_by_children(s)) { + if (action == FREEZER_FREEZE && !slice_freezer_action_supported_by_children(s)) { log_unit_warning(s, "Requested freezer operation is not supported by all children of the slice"); return 0; } @@ -386,8 +386,11 @@ static int slice_freezer_action(Unit *s, FreezerAction action) { if (action == FREEZER_FREEZE) r = UNIT_VTABLE(member)->freeze(member); - else + else if (UNIT_VTABLE(member)->thaw) r = UNIT_VTABLE(member)->thaw(member); + else + /* Thawing is requested but no corresponding method is available, ignore. */ + r = 0; if (r < 0) return r; }