]> granicus.if.org Git - sudo/commitdiff
Fix NULL dereference with "sudo -g group" when the sudoers rule has
authorTodd C. Miller <Todd.Miller@courtesan.com>
Tue, 4 Jan 2011 17:44:39 +0000 (12:44 -0500)
committerTodd C. Miller <Todd.Miller@courtesan.com>
Tue, 4 Jan 2011 17:44:39 +0000 (12:44 -0500)
no runas user or group listed.  Fixes RedHat bug Bug 667103.

plugins/sudoers/match.c

index 4a245422fdc513be1910b5fad6abdb224880eb70..0f74d61da9b436ab60c3c6d2695b8c549dd9365a 100644 (file)
@@ -165,10 +165,11 @@ _runaslist_matches(struct member_list *user_list, struct member_list *group_list
     int user_matched = UNSPEC;
     int group_matched = UNSPEC;
 
-    if (tq_empty(user_list) && tq_empty(group_list))
-       return(userpw_matches(def_runas_default, runas_pw->pw_name, runas_pw));
-
     if (runas_pw != NULL) {
+       /* If no runas user or runas group listed in sudoers, use default. */
+       if (tq_empty(user_list) && tq_empty(group_list))
+           return(userpw_matches(def_runas_default, runas_pw->pw_name, runas_pw));
+
        tq_foreach_rev(user_list, m) {
            switch (m->type) {
                case ALL: