diff options
author | Andrew G. Morgan <morgan@kernel.org> | 2009-12-25 16:08:37 -0800 |
---|---|---|
committer | Andrew G. Morgan <morgan@kernel.org> | 2009-12-25 16:09:29 -0800 |
commit | 79f8dc9636c363de0122a417b13f784c7c2d9f11 (patch) | |
tree | 51428d65c39697076ee928eacfa13d99f137be7c | |
parent | 9c49729624b1a9c2c48d43929a4cbb2ce3c8db06 (diff) | |
download | libcap-79f8dc9636c363de0122a417b13f784c7c2d9f11.tar.gz |
Oops. Don't reuse index for loop.
i -> j. The previous code would loop infinitely with
--user=<user-with-one-group> --print
Signed-off-by: Andrew G. Morgan <morgan@kernel.org>
-rw-r--r-- | progs/capsh.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/progs/capsh.c b/progs/capsh.c index 10aecf4..8f086db 100644 --- a/progs/capsh.c +++ b/progs/capsh.c @@ -434,7 +434,7 @@ int main(int argc, char *argv[], char *envp[]) printf("\n"); } else if (!strcmp("--print", argv[i])) { unsigned cap; - int set, status; + int set, status, j; cap_t all; char *text; const char *sep; @@ -490,8 +490,8 @@ int main(int argc, char *argv[], char *envp[]) printf("groups="); status = getgroups(MAX_GROUPS, groups); sep = ""; - for (i=0; i < status; i++) { - printf("%s%u", sep, groups[i]); + for (j=0; j < status; j++) { + printf("%s%u", sep, groups[j]); sep = ","; } printf("\n"); |