[Pkg-shadow-commits] r2227 - in upstream/trunk: . src

[nekral-guest at alioth.debian.org]

Author: nekral-guest
Date: 2008-07-27 22:30:47 +0000 (Sun, 27 Jul 2008)
New Revision: 2227

Modified:
   upstream/trunk/ChangeLog
   upstream/trunk/src/groupmems.c
Log:
	* src/groupmems.c: Replace an if test by an else.

Modified: upstream/trunk/ChangeLog
===================================================================
--- upstream/trunk/ChangeLog	2008-07-27 22:30:39 UTC (rev 2226)
+++ upstream/trunk/ChangeLog	2008-07-27 22:30:47 UTC (rev 2227)
@@ -1,5 +1,9 @@
 2008-07-27  Nicolas François  <nicolas.francois at centraliens.net>
 
+	* src/groupmems.c: Replace an if test by an else.
+
+2008-07-27  Nicolas François  <nicolas.francois at centraliens.net>
+
 	* src/groupmems.c: Ignore the return value of fputs().
 
 2008-07-27  Nicolas François  <nicolas.francois at centraliens.net>

Modified: upstream/trunk/src/groupmems.c
===================================================================
--- upstream/trunk/src/groupmems.c	2008-07-27 22:30:39 UTC (rev 2226)
+++ upstream/trunk/src/groupmems.c	2008-07-27 22:30:47 UTC (rev 2227)
@@ -181,9 +181,7 @@
 	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);
 	}