coreutils minor fixes for id.c, setuidgid.c

[Paul Eggert]

I installed these minor fixes.  The user-visible change is that
printing uids and gids is now supported up to ULONG_MAX, instead of
merely UINT_MAX or LONG_MAX.

2004-08-02  Paul Eggert  <address@hidden>

        * src/id.c (use_name, main, print_user, xgetgroups, print_group_list,
        print_full_info): Use bool for booleans.
        (problems): Remove, replacing with....
        (ok): New var (inverted from old sense).
        (print_user, print_group, print_full_info):
        Print uids/gids with %lu, not %u.
        (xgetgroups): Don't run out of memory if getgroups or getugroups
        returns -1.
        * src/setuidgid.c (main): Print uids/gids with %lu, not %ld.

Index: src/id.c
===================================================================
RCS file: /home/eggert/coreutils/cu/src/id.c,v
retrieving revision 1.77
diff -p -u -r1.77 id.c
--- src/id.c    21 Jun 2004 15:03:35 -0000      1.77
+++ src/id.c    20 Jul 2004 04:32:02 -0000
@@ -54,15 +54,15 @@ static void print_full_info (const char 
 /* The name this program was run with. */
 char *program_name;
 
-/* If nonzero, output user/group name instead of ID number. -n */
-static int use_name = 0;
+/* If true, output user/group name instead of ID number. -n */
+static bool use_name = false;
 
 /* The real and effective IDs of the user to print. */
 static uid_t ruid, euid;
 static gid_t rgid, egid;
 
-/* Nonzero if errors have been encountered.  */
-static int problems = 0;
+/* True unless errors have been encountered.  */
+static bool ok = true;
 
 static struct option const longopts[] =
 {
@@ -111,14 +111,14 @@ main (int argc, char **argv)
 {
   int optc;
 
-  /* If nonzero, output the list of all group IDs. -G */
-  int just_group_list = 0;
-  /* If nonzero, output only the group ID(s). -g */
-  int just_group = 0;
-  /* If nonzero, output real UID/GID instead of default effective UID/GID. -r 
*/
-  int use_real = 0;
-  /* If nonzero, output only the user ID(s). -u */
-  int just_user = 0;
+  /* If true, output the list of all group IDs. -G */
+  bool just_group_list = false;
+  /* If true, output only the group ID(s). -g */
+  bool just_group = false;
+  /* If true, output real UID/GID instead of default effective UID/GID. -r */
+  bool use_real = false;
+  /* If true, output only the user ID(s). -u */
+  bool just_user = false;
 
   initialize_main (&argc, &argv);
   program_name = argv[0];
@@ -138,19 +138,19 @@ main (int argc, char **argv)
          /* Ignore -a, for compatibility with SVR4.  */
          break;
        case 'g':
-         just_group = 1;
+         just_group = true;
          break;
        case 'n':
-         use_name = 1;
+         use_name = true;
          break;
        case 'r':
-         use_real = 1;
+         use_real = true;
          break;
        case 'u':
-         just_user = 1;
+         just_user = true;
          break;
        case 'G':
-         just_group_list = 1;
+         just_group_list = true;
          break;
        case_GETOPT_HELP_CHAR;
        case_GETOPT_VERSION_CHAR (PROGRAM_NAME, AUTHORS);
@@ -162,7 +162,7 @@ main (int argc, char **argv)
   if (just_user + just_group + just_group_list > 1)
     error (EXIT_FAILURE, 0, _("cannot print only user and only group"));
 
-  if (just_user + just_group + just_group_list == 0 && (use_real || use_name))
+  if (just_user + just_group + just_group_list == 0 && (use_real | use_name))
     error (EXIT_FAILURE, 0,
           _("cannot print only names or real IDs in default format"));
 
@@ -198,7 +198,7 @@ main (int argc, char **argv)
     print_full_info (argv[optind]);
   putchar ('\n');
 
-  exit (problems == 0 ? EXIT_SUCCESS : EXIT_FAILURE);
+  exit (ok ? EXIT_SUCCESS : EXIT_FAILURE);
 }
 
 /* Print the name or value of user ID UID. */
@@ -214,12 +214,12 @@ print_user (uid_t uid)
       if (pwd == NULL)
        {
          error (0, 0, _("cannot find name for user ID %u"), uid);
-         problems = 1;
+         ok = false;
        }
     }
 
   if (pwd == NULL)
-    printf ("%u", (unsigned) uid);
+    printf ("%lu", (unsigned long int) uid);
   else
     printf ("%s", pwd->pw_name);
 }
@@ -237,12 +237,12 @@ print_group (gid_t gid)
       if (grp == NULL)
        {
          error (0, 0, _("cannot find name for group ID %u"), gid);
-         problems = 1;
+         ok = false;
        }
     }
 
   if (grp == NULL)
-    printf ("%u", (unsigned) gid);
+    printf ("%lu", (unsigned long int) gid);
   else
     printf ("%s", grp->gr_name);
 }
@@ -251,38 +251,42 @@ print_group (gid_t gid)
 
 /* FIXME: document */
 
-static int
+static bool
 xgetgroups (const char *username, gid_t gid, int *n_groups,
            GETGROUPS_T **groups)
 {
   int max_n_groups;
   int ng;
-  GETGROUPS_T *g;
-  int fail = 0;
+  GETGROUPS_T *g = NULL;
 
-  if (username == 0)
+  if (!username)
     max_n_groups = getgroups (0, NULL);
   else
     max_n_groups = getugroups (0, NULL, username, gid);
 
-  g = xnmalloc (max_n_groups, sizeof *g);
-  if (username == 0)
-    ng = getgroups (max_n_groups, g);
+  if (max_n_groups < 0)
+    ng = -1;
   else
-    ng = getugroups (max_n_groups, g, username, gid);
-
+    {
+      g = xnmalloc (max_n_groups, sizeof *g);
+      if (!username)
+       ng = getgroups (max_n_groups, g);
+      else
+       ng = getugroups (max_n_groups, g, username, gid);
+    }
+      
   if (ng < 0)
     {
       error (0, errno, _("cannot get supplemental group list"));
-      ++fail;
-      free (groups);
+      free (g);
+      return false;
     }
-  if (!fail)
+  else
     {
       *n_groups = ng;
       *groups = g;
+      return true;
     }
-  return fail;
 }
 
 #endif /* HAVE_GETGROUPS */
@@ -296,7 +300,7 @@ print_group_list (const char *username)
 
   pwd = getpwuid (ruid);
   if (pwd == NULL)
-    problems = 1;
+    ok = false;
 
   print_group (rgid);
   if (egid != rgid)
@@ -311,10 +315,10 @@ print_group_list (const char *username)
     GETGROUPS_T *groups;
     register int i;
 
-    if (xgetgroups (username, (pwd ? pwd->pw_gid : (gid_t) -1),
-                   &n_groups, &groups))
+    if (! xgetgroups (username, (pwd ? pwd->pw_gid : (gid_t) -1),
+                     &n_groups, &groups))
       {
-       problems = 1;
+       ok = false;
        return;
       }
 
@@ -337,36 +341,36 @@ print_full_info (const char *username)
   struct passwd *pwd;
   struct group *grp;
 
-  printf ("uid=%u", (unsigned) ruid);
+  printf ("uid=%lu", (unsigned long int) ruid);
   pwd = getpwuid (ruid);
   if (pwd == NULL)
-    problems = 1;
+    ok = false;
   else
     printf ("(%s)", pwd->pw_name);
 
-  printf (" gid=%u", (unsigned) rgid);
+  printf (" gid=%lu", (unsigned long int) rgid);
   grp = getgrgid (rgid);
   if (grp == NULL)
-    problems = 1;
+    ok = false;
   else
     printf ("(%s)", grp->gr_name);
 
   if (euid != ruid)
     {
-      printf (" euid=%u", (unsigned) euid);
+      printf (" euid=%lu", (unsigned long int) euid);
       pwd = getpwuid (euid);
       if (pwd == NULL)
-       problems = 1;
+       ok = false;
       else
        printf ("(%s)", pwd->pw_name);
     }
 
   if (egid != rgid)
     {
-      printf (" egid=%u", (unsigned) egid);
+      printf (" egid=%lu", (unsigned long int) egid);
       grp = getgrgid (egid);
       if (grp == NULL)
-       problems = 1;
+       ok = false;
       else
        printf ("(%s)", grp->gr_name);
     }
@@ -377,10 +381,10 @@ print_full_info (const char *username)
     GETGROUPS_T *groups;
     register int i;
 
-    if (xgetgroups (username, (pwd ? pwd->pw_gid : (gid_t) -1),
-                   &n_groups, &groups))
+    if (! xgetgroups (username, (pwd ? pwd->pw_gid : (gid_t) -1),
+                     &n_groups, &groups))
       {
-       problems = 1;
+       ok = false;
        return;
       }
 
@@ -390,10 +394,10 @@ print_full_info (const char *username)
       {
        if (i > 0)
          putchar (',');
-       printf ("%u", (unsigned) groups[i]);
+       printf ("%lu", (unsigned long int) groups[i]);
        grp = getgrgid (groups[i]);
        if (grp == NULL)
-         problems = 1;
+         ok = false;
        else
          printf ("(%s)", grp->gr_name);
       }
Index: src/setuidgid.c
===================================================================
RCS file: /home/eggert/coreutils/cu/src/setuidgid.c,v
retrieving revision 1.10
diff -p -u -r1.10 setuidgid.c
--- src/setuidgid.c     21 Jun 2004 15:03:35 -0000      1.10
+++ src/setuidgid.c     19 Jul 2004 04:02:48 -0000
@@ -114,11 +114,11 @@ main (int argc, char **argv)
 
   if (setgid (pwd->pw_gid))
     error (SETUIDGID_FAILURE, errno,
-          _("cannot set group-ID to %ld"), (long int) pwd->pw_gid);
+          _("cannot set group-ID to %lu"), (unsigned long int) pwd->pw_gid);
 
   if (setuid (pwd->pw_uid))
     error (SETUIDGID_FAILURE, errno,
-          _("cannot set user-ID to %ld"), (long int) pwd->pw_uid);
+          _("cannot set user-ID to %lu"), (unsigned long int) pwd->pw_uid);
 
   {
     char **cmd = argv + 2;