diff options
author | Pascal Bellard | 2012-06-12 13:21:02 +0200 |
---|---|---|
committer | Denys Vlasenko | 2012-06-12 13:21:02 +0200 |
commit | 70fc8c17e2d032f34162f7abc3e65a67c0ff272a (patch) | |
tree | b2a4b1b2f6d52b01056e4601aca1c676e7f0f80f /libbb | |
parent | 588e284f53da2dc5c58f3d1b9efacd6e9061baf1 (diff) | |
download | busybox-70fc8c17e2d032f34162f7abc3e65a67c0ff272a.zip busybox-70fc8c17e2d032f34162f7abc3e65a67c0ff272a.tar.gz |
su: do not change to home dir unless -l
Signed-off-by: Pascal Bellard <pascal.bellard@ads-lu.com>
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'libbb')
-rw-r--r-- | libbb/setup_environment.c | 8 | ||||
-rw-r--r-- | libbb/xfuncs_printf.c | 4 |
2 files changed, 7 insertions, 5 deletions
diff --git a/libbb/setup_environment.c b/libbb/setup_environment.c index 73229ca..4258656 100644 --- a/libbb/setup_environment.c +++ b/libbb/setup_environment.c @@ -37,9 +37,11 @@ void FAST_FUNC setup_environment(const char *shell, int flags, const struct pass /* Change the current working directory to be the home directory * of the user */ - if (chdir(pw->pw_dir)) { - xchdir((flags & SETUP_ENV_TO_TMP) ? "/tmp" : "/"); - bb_error_msg("can't chdir to home directory '%s'", pw->pw_dir); + if (!(flags & SETUP_ENV_NO_CHDIR)) { + if (chdir(pw->pw_dir) != 0) { + bb_error_msg("can't change directory to '%s'", pw->pw_dir); + xchdir((flags & SETUP_ENV_TO_TMP) ? "/tmp" : "/"); + } } if (flags & SETUP_ENV_CLEARENV) { diff --git a/libbb/xfuncs_printf.c b/libbb/xfuncs_printf.c index d8a42ba..05aa07c 100644 --- a/libbb/xfuncs_printf.c +++ b/libbb/xfuncs_printf.c @@ -355,13 +355,13 @@ void FAST_FUNC xsetuid(uid_t uid) void FAST_FUNC xchdir(const char *path) { if (chdir(path)) - bb_perror_msg_and_die("chdir(%s)", path); + bb_perror_msg_and_die("can't change directory to '%s'", path); } void FAST_FUNC xchroot(const char *path) { if (chroot(path)) - bb_perror_msg_and_die("can't change root directory to %s", path); + bb_perror_msg_and_die("can't change root directory to '%s'", path); xchdir("/"); } |