id: better wording in comments, no code changes

1 files changed, 6 insertions, 6 deletions

diff --git a/coreutils/id.c b/coreutils/id.c
index b00532f83..001595f67 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)