d/freebsd-nq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

The call to setuid(2) subsequently causes setgroups(2) to fail. setgroups(2)

Browse Source 
requires super-user access in order to complete successfully.
Move setgroups(2) to execute before setuid(2) so that it is successful.
This commit is contained in:
Christian S.J. Peron 2004-06-30 18:58:19 +00:00
parent f5298929e3
commit 0c90b4887e
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
libexec/tftpd/tftpd.c
View File

@ -286,9 +286,9 @@ main(int argc, char *argv[])
syslog(LOG_ERR, "chroot: %s: %m", chroot_dir);
exit(1);
}
chdir( "/" );
setuid(nobody->pw_uid);
chdir("/");
setgroups(1, &nobody->pw_gid);
setuid(nobody->pw_uid);
}
len = sizeof(me);