From a18ae861e77f106517df203e30a810ba59b082d9 Mon Sep 17 00:00:00 2001 From: liupan <490021209@qq.com> Date: Wed, 6 Jul 2022 13:53:35 -0600 Subject: [PATCH] api.c: avoid ambiguous controller name matches calling "hasmntopt" to determine if the controller name exists in "mntopt", may cause errors because of "hasmntopt" only match substring. cpu controller may incorrectly match to cpuset when cpuset mount info appeared before cpu,cpuacct in "/proc/mounts", so we need to validate the last character to make sure the controller name matches exactly. Signed-off-by: liupan <490021209@qq.com> Reviewed-by: Kamalesh Babulal Signed-off-by: Tom Hromatka --- src/api.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/src/api.c b/src/api.c index 86a57e04..39fcbe6f 100644 --- a/src/api.c +++ b/src/api.c @@ -1135,6 +1135,7 @@ STATIC int cgroup_process_v1_mnt(char *controllers[], struct mntent *ent, char *strtok_buffer = NULL, *mntopt = NULL; int shared_mnt, duplicate; int i, j, ret = 0; + char c = 0; for (i = 0; controllers[i] != NULL; i++) { mntopt = hasmntopt(ent, controllers[i]); @@ -1142,6 +1143,11 @@ STATIC int cgroup_process_v1_mnt(char *controllers[], struct mntent *ent, if (!mntopt) continue; + c = mntopt[strlen(controllers[i])]; + + if (c != '\0' && c != ',') + continue; + cgroup_dbg("found %s in %s\n", controllers[i], ent->mnt_opts); /* Check if controllers share mount points */ -- 2.47.2