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

Avoid comparing new cwd with old one if getcwd() failed.

Browse Source 
Bug #670
This commit is contained in:
Todd C. Miller
2014-10-07 10:56:50 -06:00
parent 1890f0fcd6
commit dc67442151
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/sudo.c
View File

@@ -1003,8 +1003,9 @@ exec_setup(struct command_details *details, const char *ptyname, int ptyfd)
* Only change cwd if we have chroot()ed or the policy modules * Only change cwd if we have chroot()ed or the policy modules
* specifies a different cwd. Must be done after uid change. * specifies a different cwd. Must be done after uid change.
*/ */
if (details->cwd) { if (details->cwd != NULL) {
if (details->chroot || strcmp(details->cwd, user_details.cwd) != 0) { if (details->chroot || user_details.cwd == NULL ||
strcmp(details->cwd, user_details.cwd) != 0) {
/* Note: cwd is relative to the new root, if any. */ /* Note: cwd is relative to the new root, if any. */
if (chdir(details->cwd) != 0) { if (chdir(details->cwd) != 0) {
sudo_warn(U_("unable to change directory to %s"), details->cwd); sudo_warn(U_("unable to change directory to %s"), details->cwd);