diff --git a/src/basic/cgroup-util.c b/src/basic/cgroup-util.c index 7dd8e4e0fc6..9ba9f375b05 100644 --- a/src/basic/cgroup-util.c +++ b/src/basic/cgroup-util.c @@ -2001,7 +2001,7 @@ int cg_mask_supported(CGroupMask *ret) { } int cg_kernel_controllers(Set **ret) { - _cleanup_set_free_free_ Set *controllers = NULL; + _cleanup_set_free_ Set *controllers = NULL; _cleanup_fclose_ FILE *f = NULL; int r; @@ -2011,10 +2011,6 @@ int cg_kernel_controllers(Set **ret) { * and controllers that aren't currently accessible (because not mounted). This does not include "name=" * pseudo-controllers. */ - controllers = set_new(&string_hash_ops); - if (!controllers) - return -ENOMEM; - r = fopen_unlocked("/proc/cgroups", "re", &f); if (r == -ENOENT) { *ret = NULL; @@ -2048,7 +2044,7 @@ int cg_kernel_controllers(Set **ret) { if (!cg_controller_is_valid(controller)) return -EBADMSG; - r = set_consume(controllers, TAKE_PTR(controller)); + r = set_ensure_consume(&controllers, &string_hash_ops_free, TAKE_PTR(controller)); if (r < 0) return r; } diff --git a/src/nspawn/nspawn-cgroup.c b/src/nspawn/nspawn-cgroup.c index cb01b25bc67..d472e80c035 100644 --- a/src/nspawn/nspawn-cgroup.c +++ b/src/nspawn/nspawn-cgroup.c @@ -406,7 +406,7 @@ static int mount_legacy_cgns_unsupported( uid_t uid_range, const char *selinux_apifs_context) { - _cleanup_set_free_free_ Set *controllers = NULL; + _cleanup_set_free_ Set *controllers = NULL; const char *cgroup_root; int r; diff --git a/src/shared/mount-setup.c b/src/shared/mount-setup.c index 3b309828351..0f2fb5d098a 100644 --- a/src/shared/mount-setup.c +++ b/src/shared/mount-setup.c @@ -292,7 +292,7 @@ static int symlink_controller(const char *target, const char *alias) { } int mount_cgroup_controllers(void) { - _cleanup_set_free_free_ Set *controllers = NULL; + _cleanup_set_free_ Set *controllers = NULL; int r; if (!cg_is_legacy_wanted())