summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDenis Vlasenko2008-10-29 10:30:54 +0000
committerDenis Vlasenko2008-10-29 10:30:54 +0000
commite7368f16fac4ace9474090c9d1f58a145db92650 (patch)
tree2921b5f27c978f163ba61cc00ea3b2a7327abf8d
parent58f3c2e01ee5c5d7f70ee3701b160c0e9d6e6e52 (diff)
downloadbusybox-e7368f16fac4ace9474090c9d1f58a145db92650.zip
busybox-e7368f16fac4ace9474090c9d1f58a145db92650.tar.gz
id: better wording in comments, no code changes
-rw-r--r--coreutils/id.c12
1 files changed, 6 insertions, 6 deletions
diff --git a/coreutils/id.c b/coreutils/id.c
index b00532f..001595f 100644
--- a/coreutils/id.c
+++ b/coreutils/id.c
@@ -84,7 +84,7 @@ static int get_groups(const char *username, gid_t rgid, gid_t *groups, int *n)
//if (*n < 0)
// return 0;
//return m;
- //commented here, happens below anyway
+ //commented out here, happens below anyway
} else {
/* On error -1 is returned, which ends up in *n */
int nn = getgroups(*n, groups);
@@ -152,9 +152,9 @@ int id_main(int argc UNUSED_PARAM, char **argv)
if (egid != rgid)
status |= print_group(egid, " ");
}
- /* We'd rather try supplying largish buffer than
- * having get_groups() run twice. That might be slow
- * (think about "user database in remove SQL server" case) */
+ /* We are supplying largish buffer, trying
+ * to not run get_groups() twice. That might be slow
+ * ("user database in remote SQL server" case) */
groups = xmalloc(64 * sizeof(gid_t));
n = 64;
if (get_groups(username, rgid, groups, &n) < 0) {
@@ -171,14 +171,14 @@ int id_main(int argc UNUSED_PARAM, char **argv)
status |= print_group(groups[i], opt ? " " : prefix);
prefix = ",";
}
- if (ENABLE_FEATURE_CLEAN_UP)
- free(groups);
} else if (n < 0) { /* error in get_groups() */
if (!ENABLE_DESKTOP)
bb_error_msg_and_die("cannot get groups");
else
return EXIT_FAILURE;
}
+ if (ENABLE_FEATURE_CLEAN_UP)
+ free(groups);
#if ENABLE_SELINUX
if (is_selinux_enabled()) {
if (getcon(&scontext) == 0)