From c46591c5cf0d85b551668d98b33c86a68853b3d1 Mon Sep 17 00:00:00 2001 From: Kamalesh Babulal Date: Wed, 13 Jul 2022 13:42:11 -0600 Subject: [PATCH] api.c: add ret value fix in cgroup_change_cgroup_flags() MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Fix the unused ret value warning, reported by Coverity: CID 258304 (#1 of 1): Unused value (UNUSED_VALUE)returned_value: Assigning value from cgroup_create_template_group(newdest, tmp, flags) to ret here, but that stored value is overwritten before it can be used. It turns out that cgroup_change_cgroup_flags(), calls the function cgroup_create_template_group() to a cgroup based on a template, which may fail and we don't check the return value for the error.  This fix adds the check and bails out on error. Signed-off-by: Kamalesh Babulal Signed-off-by: Tom Hromatka --- src/api.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/src/api.c b/src/api.c index f243ed1f..fa4a3eda 100644 --- a/src/api.c +++ b/src/api.c @@ -4208,6 +4208,11 @@ int cgroup_change_cgroup_flags(uid_t uid, gid_t gid, const char *procname, pid_t cgroup_dbg("control group %s is template\n", newdest); ret = cgroup_create_template_group(newdest, tmp, flags); + if (ret) { + cgroup_warn("failed to create cgroup based on template %s\n", + newdest); + goto finished; + } } /* Apply the rule */ -- 2.47.2