(statfs): Use stat, not safe_stat.

[gnulib.git] / lib / userspec.c

diff --git a/lib/userspec.c b/lib/userspec.c
index 34b1d32..67f1583 100644 (file)
--- a/lib/userspec.c
+++ b/lib/userspec.c
@@ -18,13 +18,20 @@
 /* Written by David MacKenzie <djm@gnu.ai.mit.edu>.  */
 \f
 #ifdef HAVE_CONFIG_H
-#if defined (CONFIG_BROKETS)
-/* We use <config.h> instead of "config.h" so that a compilation
-   using -I. -I$srcdir will use ./config.h rather than $srcdir/config.h
-   (which it would do because it found this file in $srcdir).  */
 #include <config.h>
+#endif
+
+#ifdef __GNUC__
+#define alloca __builtin_alloca
+#else
+#ifdef HAVE_ALLOCA_H
+#include <alloca.h>
 #else
-#include "config.h"
+#ifdef _AIX
+ #pragma alloca
+#else
+char *alloca ();
+#endif
 #endif
 #endif
 
@@ -44,8 +51,6 @@
 
 #ifdef STDC_HEADERS
 #include <stdlib.h>
-#else
-char *malloc ();
 #endif
 
 #ifdef HAVE_UNISTD_H
@@ -63,6 +68,18 @@ struct group *getgrgid ();
 #define endgrent()
 #endif
 
+/* Perform the equivalent of the statement `dest = strdup (src);',
+   but obtaining storage via alloca instead of from the heap.  */
+
+#define V_STRDUP(dest, src)                                            \
+  do                                                                   \
+    {                                                                  \
+      int _len = strlen ((src));                                       \
+      (dest) = (char *) alloca (_len + 1);                             \
+      strcpy (dest, src);                                              \
+    }                                                                  \
+  while (0)
+
 #define isdigit(c) ((c) >= '0' && (c) <= '9')
 
 char *strdup ();
@@ -93,26 +110,25 @@ isnumber (str)
    Return NULL if successful, a static error message string if not.  */
 
 const char *
-parse_user_spec (spec_arg, uid, gid, username, groupname)
+parse_user_spec (spec_arg, uid, gid, username_arg, groupname_arg)
      const char *spec_arg;
      uid_t *uid;
      gid_t *gid;
-     char **username, **groupname;
+     char **username_arg, **groupname_arg;
 {
   static const char *tired = "virtual memory exhausted";
   const char *error_msg;
   char *spec;                  /* A copy we can write on.  */
-  int spec_len;
   struct passwd *pwd;
   struct group *grp;
   char *g, *u, *separator;
+  char *groupname;
 
   error_msg = NULL;
-  *username = *groupname = NULL;
+  *username_arg = *groupname_arg = NULL;
+  groupname = NULL;
 
-  spec_len = strlen (spec_arg);
-  spec = (char *) alloca (strlen (spec_arg) + 1);
-  strcpy (spec, spec_arg);
+  V_STRDUP (spec, spec_arg);
 
   /* Find the separator if there is one.  */
   separator = index (spec, ':');
@@ -168,28 +184,19 @@ parse_user_spec (spec_arg, uid, gid, username, groupname)
                     zero byte.  */
                  char uint_buf[21];
                  sprintf (uint_buf, "%u", (unsigned) (pwd->pw_gid));
-                 *groupname = strdup (uint_buf);
+                 V_STRDUP (groupname, uint_buf);
                }
              else
                {
-                 *groupname = strdup (grp->gr_name);
+                 V_STRDUP (groupname, grp->gr_name);
                }
-             if (*groupname == NULL)
-               error_msg = tired;
              endgrent ();
            }
        }
       endpwent ();
-
-      if (error_msg == NULL)
-       {
-         *username = strdup (u);
-         if (*username == NULL)
-           error_msg = tired;
-       }
     }
 
-  if (g != NULL)
+  if (g != NULL && error_msg == NULL)
     {
       /* Explicit group.  */
       grp = getgrnam (g);
@@ -205,31 +212,37 @@ parse_user_spec (spec_arg, uid, gid, username, groupname)
       endgrent ();             /* Save a file descriptor.  */
 
       if (error_msg == NULL)
-       {
-         *groupname = strdup (g);
-         if (*groupname == NULL)
-           return tired;
-       }
+       V_STRDUP (groupname, g);
     }
 
-  if (error_msg)
+  if (error_msg == NULL)
     {
-      if (*groupname != NULL)
+      if (u != NULL)
        {
-         free (*groupname);
-         *groupname = NULL;
+         *username_arg = strdup (u);
+         if (*username_arg == NULL)
+           error_msg = tired;
        }
-      if (*username != NULL)
+
+      if (groupname != NULL && error_msg == NULL)
        {
-         free (*username);
-         *username = NULL;
+         *groupname_arg = strdup (groupname);
+         if (*groupname_arg == NULL)
+           {
+             if (*username_arg != NULL)
+               {
+                 free (*username_arg);
+                 *username_arg = NULL;
+               }
+             error_msg = tired;
+           }
        }
     }
 
   return error_msg;
 }
 
-#ifdef TESTING
+#ifdef TEST
 
 #define NULL_CHECK(s) ((s) == NULL ? "(null)" : (s))