* src/groupmems.c: Replace an if test by an else.

diff --git a/ChangeLog b/ChangeLog
index 1b55c5473043f16f86b0db6b5e5644c4f9d02892..6d15f22f4a4530511b0d4f853003b7e22156c638 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2008-07-27  Nicolas François  <nicolas.francois@centraliens.net>
+
+       * src/groupmems.c: Replace an if test by an else.
+
 2008-07-27  Nicolas François  <nicolas.francois@centraliens.net>
 
        * src/groupmems.c: Ignore the return value of fputs().

diff --git a/src/groupmems.c b/src/groupmems.c
index 246585726d35dad8d274c0597029fe283073bcf7..3c7facdcb2528a28ade49a2b1753aa439d0369b1 100644 (file)
--- a/src/groupmems.c
+++ b/src/groupmems.c
@@ -181,9 +181,7 @@ static void check_perms (void)
        pampw = getpwuid (getuid ()); /* local, no need for xgetpwuid */
        if (NULL == pampw) {
                retval = PAM_USER_UNKNOWN;
-       }
-
-       if (PAM_SUCCESS == retval) {
+       } else {
                retval = pam_start ("groupmod", pampw->pw_name,
                                    &conv, &pamh);
        }