From: nekral-guest <nekral-guest@5a98b0ae-9ef6-0310-add3-de5d479b70d7>
Date: Thu, 14 Jul 2011 13:29:27 +0000 (+0000)
Subject: 	* src/usermod.c (update_gshadow): is_member was computed twice.
X-Git-Tag: 4.1.5~162
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=8195a2b5d804efdb82d67e31a8451daf7d66e53e;p=shadow

	* src/usermod.c (update_gshadow): is_member was computed twice.
---

diff --git a/ChangeLog b/ChangeLog
index aef33d88..3ff52799 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -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@centraliens.net>
 
diff --git a/src/usermod.c b/src/usermod.c
index 34e279b2..4d7d0988 100644
--- a/src/usermod.c
+++ b/src/usermod.c
@@ -699,7 +699,6 @@ static void update_gshadow (void)
 		 * 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));