X-Git-Url: http://erislabs.net/gitweb/?a=blobdiff_plain;f=lib%2Fuserspec.c;h=67f1583335c513b3043a01a83e5484bdf6644f6c;hb=d33ed582fc804afef36639a9fbcf8a3e242dc7b7;hp=6de8b824d055cb783a1cde4457a8506ccc1ce96a;hpb=5a84bc4587b2b38448098dd7f69843a83ecd24dc;p=gnulib.git diff --git a/lib/userspec.c b/lib/userspec.c index 6de8b824d..67f158333 100644 --- a/lib/userspec.c +++ b/lib/userspec.c @@ -18,13 +18,20 @@ /* Written by David MacKenzie . */ #ifdef HAVE_CONFIG_H -#if defined (CONFIG_BROKETS) -/* We use 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 +#endif + +#ifdef __GNUC__ +#define alloca __builtin_alloca +#else +#ifdef HAVE_ALLOCA_H +#include #else -#include "config.h" +#ifdef _AIX + #pragma alloca +#else +char *alloca (); +#endif #endif #endif @@ -61,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 (); @@ -91,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, ':'); @@ -166,25 +184,16 @@ 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 && error_msg == NULL) @@ -203,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) - error_msg = 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))