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

[Nicolas FRANÇOIS]

Author: nekral-guest
Date: 2011-07-14 13:29:27 +0000 (Thu, 14 Jul 2011)
New Revision: 3388

Modified:
   upstream/trunk/ChangeLog
   upstream/trunk/src/usermod.c
Log:
	* src/usermod.c (update_gshadow): is_member was computed twice.

Modified: upstream/trunk/ChangeLog
===================================================================
--- upstream/trunk/ChangeLog	2011-07-14 13:29:22 UTC (rev 3387)
+++ upstream/trunk/ChangeLog	2011-07-14 13:29:27 UTC (rev 3388)
@@ -12,6 +12,7 @@
 	* src/usermod.c (process_flags): Check for oflg is not needed to
 	check if changes are needed.
 	* src/usermod.c: usage() does not return. Add annotations.
+	* src/usermod.c (update_gshadow): is_member was computed twice.
 
 2011-07-08  Nicolas François  <nicolas.francois at centraliens.net>
 

Modified: upstream/trunk/src/usermod.c
===================================================================
--- upstream/trunk/src/usermod.c	2011-07-14 13:29:22 UTC (rev 3387)
+++ upstream/trunk/src/usermod.c	2011-07-14 13:29:27 UTC (rev 3388)
@@ -699,7 +699,6 @@
 		 * See if the user specified this group as one of their
 		 * concurrent groups.
 		 */
-		is_member = Gflg && is_on_list (user_groups, sgrp->sg_name);
 		is_member = Gflg && (   (was_member && aflg)
 		                     || is_on_list (user_groups, sgrp->sg_name));