diff options
author | Nicholas Marriott <nicm@openbsd.org> | 2012-05-28 08:55:43 +0000 |
---|---|---|
committer | Nicholas Marriott <nicm@openbsd.org> | 2012-05-28 08:55:43 +0000 |
commit | f51c55c3f032e4c525cc7aa7ac74c1ce4a9d29ab (patch) | |
tree | c0e275c09d1c066a4a4f4da919677148408282e8 | |
parent | fbf2bd865cb2ea19572a259f2011f37975e41647 (diff) | |
download | rtmux-f51c55c3f032e4c525cc7aa7ac74c1ce4a9d29ab.tar.gz rtmux-f51c55c3f032e4c525cc7aa7ac74c1ce4a9d29ab.tar.bz2 rtmux-f51c55c3f032e4c525cc7aa7ac74c1ce4a9d29ab.zip |
Use default-shell not _PATH_BSHELL to spawn commands, pointed out by
Dennis G?nnewig and Thomas Adam.
-rw-r--r-- | window.c | 17 |
1 files changed, 9 insertions, 8 deletions
@@ -739,23 +739,24 @@ window_pane_spawn(struct window_pane *wp, const char *cmd, const char *shell, clear_signals(1); log_close(); - if (*wp->cmd != '\0') { - /* Set SHELL but only if it is currently not useful. */ - shell = getenv("SHELL"); - if (checkshell(shell)) - setenv("SHELL", wp->shell, 1); + setenv("SHELL", wp->shell, 1); + ptr = strrchr(wp->shell, '/'); - execl(_PATH_BSHELL, "sh", "-c", wp->cmd, (char *) NULL); + if (*wp->cmd != '\0') { + /* Use the command. */ + if (ptr != NULL && *(ptr + 1) != '\0') + xasprintf(&argv0, "%s", ptr + 1); + else + xasprintf(&argv0, "%s", wp->shell); + execl(wp->shell, argv0, "-c", wp->cmd, (char *) NULL); fatal("execl failed"); } /* No command; fork a login shell. */ - ptr = strrchr(wp->shell, '/'); if (ptr != NULL && *(ptr + 1) != '\0') xasprintf(&argv0, "-%s", ptr + 1); else xasprintf(&argv0, "-%s", wp->shell); - setenv("SHELL", wp->shell, 1); execl(wp->shell, argv0, (char *) NULL); fatal("execl failed"); } |