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

Fixed NFS-mounted sudoers file under solaris both uid *and* gid

Browse Source 
were being set to -2.  Now beat NFS to the punch and set uid
to "nobody" ourselves, preserving group 0 to read sudoers.
This commit is contained in:
Todd C. Miller
1996-06-25 22:02:42 +00:00
parent d60779d01b
commit ea3da19711
1 changed files with 26 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

32
sudo.c
View File

@@ -305,6 +305,7 @@ int main(argc, argv)
/* validate the user but don't search for "validate" */ /* validate the user but don't search for "validate" */
rtn = validate((sudo_mode != MODE_VALIDATE)); rtn = validate((sudo_mode != MODE_VALIDATE));
switch (rtn) { switch (rtn) {
case VALIDATE_OK: case VALIDATE_OK:
@@ -815,6 +816,8 @@ void set_perms(perm)
break; break;
case PERM_USER: case PERM_USER:
(void) setgid(user_gid);
if (seteuid(user_uid)) { if (seteuid(user_uid)) {
perror("seteuid(user_uid)"); perror("seteuid(user_uid)");
exit(1); exit(1);
@@ -827,8 +830,10 @@ void set_perms(perm)
exit(1); exit(1);
} }
(void) setgid(user_gid);
if (setuid(user_uid)) { if (setuid(user_uid)) {
perror("setuid(uid)"); perror("setuid(user_uid)");
exit(1); exit(1);
} }
@@ -880,14 +885,29 @@ void set_perms(perm)
exit(1); exit(1);
} }
if (seteuid(SUDOERS_UID)) { if (setgid(SUDOERS_GID)) {
perror("seteuid(SUDOERS_UID)"); perror("setgid(SUDOERS_GID)");
exit(1); exit(1);
} }
if (setegid(SUDOERS_GID)) { /*
perror("setegid(SUDOERS_GID)"); * If SUDOERS_UID == 0 we need to use
exit(1); * a different uid in order to avoid
* NFS lossage. Sigh.
*/
if (SUDOERS_UID) {
if (seteuid(SUDOERS_UID)) {
perror("seteuid(SUDOERS_UID)");
exit(1);
}
} else {
if (!(pw_ent = getpwnam("nobody")))
pw_ent->pw_uid = (uid_t) -2;
if (seteuid(pw_ent->pw_uid)) {
perror("seteuid(nobody)");
exit(1);
}
} }
break; break;