patch from upstream to fix special case in password checking code
authorBdale Garbee <bdale@gag.com>
Tue, 11 Jan 2011 17:18:15 +0000 (10:18 -0700)
committerBdale Garbee <bdale@gag.com>
Tue, 11 Jan 2011 17:18:15 +0000 (10:18 -0700)
when only the gid is changing, closes #609641

check.c

diff --git a/check.c b/check.c
index afb6c2267dd9912cb127497e973cb790f4deff78..badf3e4007735df5ec8ad5e454bd3cefc811cf9b 100644 (file)
--- a/check.c
+++ b/check.c
@@ -119,7 +119,13 @@ check_user(validated, mode)
     if (ISSET(mode, MODE_INVALIDATE)) {
        SET(validated, FLAG_CHECK_USER);
     } else {
-       if (user_uid == 0 || user_uid == runas_pw->pw_uid || user_is_exempt())
+       /*
+        * Don't prompt for the root passwd or if the user is exempt.
+        * If the user is not changing uid/gid, no need for a password.
+        */
+       if (user_uid == 0 || (user_uid == runas_pw->pw_uid &&
+           (!runas_gr || user_in_group(sudo_user.pw, runas_gr->gr_name))) ||
+           user_is_exempt())
            return;
     }