X-Git-Url: http://erislabs.net/gitweb/?a=blobdiff_plain;f=lib%2Fgetugroups.c;h=b266bb8c320108144069dcc1066c007224da52bf;hb=0add5d71522a4be3ec75152f22b4f61aeaf1d1b2;hp=d63a6072e4861227f838f4d8aa60dd7eb1a71f6f;hpb=22341687ef7e96116b9388b25280736d527660b3;p=gnulib.git
diff --git a/lib/getugroups.c b/lib/getugroups.c
index d63a6072e..b266bb8c3 100644
--- a/lib/getugroups.c
+++ b/lib/getugroups.c
@@ -1,11 +1,11 @@
/* getugroups.c -- return a list of the groups a user is in
- Copyright (C) 1990, 1991, 1998-2000, 2003-2007 Free Software Foundation.
+ Copyright (C) 1990, 1991, 1998-2000, 2003-2008 Free Software Foundation.
- This program is free software; you can redistribute it and/or modify
+ This program is free software: you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
- the Free Software Foundation; either version 2, or (at your option)
- any later version.
+ the Free Software Foundation; either version 3 of the License, or
+ (at your option) any later version.
This program is distributed in the hope that it will be useful,
but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ -13,8 +13,7 @@
GNU General Public License for more details.
You should have received a copy of the GNU General Public License
- along with this program; if not, write to the Free Software Foundation,
- Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. */
+ along with this program. If not, see . */
/* Written by David MacKenzie. */
@@ -22,20 +21,18 @@
#include "getugroups.h"
+#include
#include /* grp.h on alpha OSF1 V2.0 uses "FILE *". */
#include
#include
#include
-#ifndef EOVERFLOW
-# define EOVERFLOW EINVAL
-#endif
/* Some old header files might not declare setgrent, getgrent, and endgrent.
If you don't have them at all, we can't implement this function.
You lose! */
-struct group *getgrent ();
+struct group *getgrent (void);
#include
@@ -45,14 +42,13 @@ struct group *getgrent ();
process. Store at most MAXCOUNT group IDs in the GROUPLIST array.
If GID is not -1, store it first (if possible). GID should be the
group ID (pw_gid) obtained from getpwuid, in case USERNAME is not
- listed in /etc/groups.
- Always return the number of groups of which USERNAME is a member. */
+ listed in /etc/groups. Upon failure, set errno and return -1.
+ Otherwise, return the number of IDs we've written into GROUPLIST. */
int
getugroups (int maxcount, GETGROUPS_T *grouplist, char const *username,
gid_t gid)
{
- struct group *grp;
int count = 0;
if (gid != (gid_t) -1)
@@ -63,9 +59,16 @@ getugroups (int maxcount, GETGROUPS_T *grouplist, char const *username,
}
setgrent ();
- while ((grp = getgrent ()) != 0)
+ while (1)
{
char **cp;
+ struct group *grp;
+
+ errno = 0;
+ grp = getgrent ();
+ if (grp == NULL)
+ break;
+
for (cp = grp->gr_mem; *cp; ++cp)
{
int n;
@@ -87,19 +90,25 @@ getugroups (int maxcount, GETGROUPS_T *grouplist, char const *username,
goto done;
grouplist[count] = grp->gr_gid;
}
- count++;
- if (count < 0)
+ if (count == INT_MAX)
{
errno = EOVERFLOW;
- count = -1;
goto done;
}
+ count++;
}
}
}
+ if (errno != 0)
+ count = -1;
+
done:
- endgrent ();
+ {
+ int saved_errno = errno;
+ endgrent ();
+ errno = saved_errno;
+ }
return count;
}