diff options
author | Ken'ichi Ohmichi <oomichi@mxs.nes.nec.co.jp> | 2009-06-25 09:12:00 +0900 |
---|---|---|
committer | Dhaval Giani <dhaval@linux.vnet.ibm.com> | 2009-06-29 16:53:21 +0530 |
commit | 9e9741892aeffc3ec7974af259c258821b073e8a (patch) | |
tree | 50d39b11d6034cfa162bd40131f423fe2ab663bf | |
parent | 81dda1ea06a62fb2db52af5c026ae196a2570702 (diff) | |
download | libcg-9e9741892aeffc3ec7974af259c258821b073e8a.tar.gz libcg-9e9741892aeffc3ec7974af259c258821b073e8a.tar.xz libcg-9e9741892aeffc3ec7974af259c258821b073e8a.zip |
Cleanup: Clarify the number meanings of string length.
Hi,
This patch clarifies the number meanings of string length in the function
cgroup_get_uid_gid_from_procfs(). This is recreated based on Igawa-san's
patch for the latest code.
* Reference
Igawa-san's patch
http://sourceforge.net/mailarchive/message.php?msg_name=20090515.085922.109268324.igawa%40mxs.nes.nec.co.jp
Signed-off-by: Masayuki Igawa <igawa@mxs.nes.nec.co.jp>
Signed-off-by: Ken'ichi Ohmichi <oomichi@mxs.nes.nec.co.jp>
Signed-off-by: Dhaval Giani <dhaval@linux.vnet.ibm.com>
-rw-r--r-- | src/api.c | 8 |
1 files changed, 4 insertions, 4 deletions
@@ -2720,15 +2720,15 @@ int cgroup_get_uid_gid_from_procfs(pid_t pid, uid_t *euid, gid_t *egid) while (fgets(buf, sizeof(buf), f)) { if (!strncmp(buf, "Uid:", 4)) { - if (sscanf((buf + 5), "%d%d%d%d", &ruid, euid, - &suid, &fsuid) != 4) + if (sscanf((buf + strlen("Uid:") + 1), "%d%d%d%d", + &ruid, euid, &suid, &fsuid) != 4) break; cgroup_dbg("Scanned proc values are %d %d %d %d\n", ruid, *euid, suid, fsuid); found_euid = true; } else if (!strncmp(buf, "Gid:", 4)) { - if (sscanf((buf + 5), "%d%d%d%d", &rgid, egid, - &sgid, &fsgid) != 4) + if (sscanf((buf + strlen("Gid:") + 1), "%d%d%d%d", + &rgid, egid, &sgid, &fsgid) != 4) break; cgroup_dbg("Scanned proc values are %d %d %d %d\n", rgid, *egid, sgid, fsgid); |