isa/sudo
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Handle sudo_get_grlist() returning NULL which can happen if

Browse Source 
getgrouplist() fails even after allocating the appropriate amount
of memory.  From Stephane Chazelas
This commit is contained in:
Todd C. Miller
2015-02-05 11:17:24 -07:00
parent 6d71b488b6
commit b727d4309c
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
plugins/sudoers/set_perms.c
View File

@@ -1589,9 +1589,11 @@ runas_setgroups(void)
#ifdef HAVE_SETAUTHDB #ifdef HAVE_SETAUTHDB
aix_restoreauthdb(); aix_restoreauthdb();
#endif #endif
if (sudo_setgroups(grlist->ngids, grlist->gids) < 0) { if (grlist != NULL) {
sudo_grlist_delref(grlist); if (sudo_setgroups(grlist->ngids, grlist->gids) < 0) {
grlist = NULL; sudo_grlist_delref(grlist);
grlist = NULL;
}
} }
debug_return_ptr(grlist); debug_return_ptr(grlist);
} }