Handle sudo_get_grlist() returning NULL which can happen if

getgrouplist() fails even after allocating the appropriate amount
of memory.  From Stephane Chazelas

diff --git a/plugins/sudoers/set_perms.c b/plugins/sudoers/set_perms.c
index 76234af355fb544fb9ea25ada119d19592137e06..6b073c7e66f647bb5d0950a598facea1cd3532a0 100644 (file)
--- a/plugins/sudoers/set_perms.c
+++ b/plugins/sudoers/set_perms.c
@@ -1589,9 +1589,11 @@ runas_setgroups(void)
 #ifdef HAVE_SETAUTHDB
     aix_restoreauthdb();
 #endif
-    if (sudo_setgroups(grlist->ngids, grlist->gids) < 0) {
-       sudo_grlist_delref(grlist);
-       grlist = NULL;
+    if (grlist != NULL) {
+       if (sudo_setgroups(grlist->ngids, grlist->gids) < 0) {
+           sudo_grlist_delref(grlist);
+           grlist = NULL;
+       }
     }
     debug_return_ptr(grlist);
 }