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

diff --git a/ChangeLog b/ChangeLog
index aef33d8813c5ce3798853a446459265f88aaf7d8..3ff5279946dc6bb9e5f609ce84ae44660e9b7b18 100644 (file)
--- 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 34e279b2465f9cfa6a783b41612f50f3c5d2f789..4d7d0988d71f663b3263df840345ba0c9e5a6ec5 100644 (file)
--- 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));