* src/groupmems.c: Remove isgroup(), which always returns TRUE.

diff --git a/ChangeLog b/ChangeLog
index b0fcd392c01e82f6f0ffa28626caf7f06d988c3f..e42ed1f13a477b8846e852405f29c256ad29229d 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2008-07-27  Nicolas François  <nicolas.francois@centraliens.net>
+
+       * src/groupmems.c: Remove isgroup(), which always returns TRUE.
+
 2008-07-27  Nicolas François  <nicolas.francois@centraliens.net>
 
        * src/groupmems.c: Reuse the functions from libmisc/list.c to deal

diff --git a/src/groupmems.c b/src/groupmems.c
index 2056312ff6e17512319455968eeea3de74d41c55..e3922861d310f020332713824f7a3d573af9c568 100644 (file)
--- a/src/groupmems.c
+++ b/src/groupmems.c
@@ -71,14 +71,6 @@ static char *Prog;
 
 #define isroot()               (getuid () == 0)
 
-static int isgroup (void)
-{
-       gid_t g = getgid ();
-       struct group *grp = getgrgid (g); /* local, no need for xgetgrgid */
-
-       return TRUE;
-}
-
 static char *whoami (void)
 {
        /* local, no need for xgetgrgid */
@@ -189,9 +181,6 @@ int main (int argc, char **argv)
                exit (EXIT_NOT_ROOT);
        } else if (isroot () && NULL != thisgroup) {
                name = thisgroup;
-       } else if (!isgroup ()) {
-               fputs (_("Group access is required\n"), stderr);
-               exit (EXIT_NOT_EROOT);
        } else if (NULL == (name = whoami ())) {
                fputs (_("Not primary owner of current group\n"), stderr);
                exit (EXIT_NOT_PRIMARY);