- don't free user-supplied string (via -e)

[oweals/busybox.git] / libbb / run_shell.c

diff --git a/libbb/run_shell.c b/libbb/run_shell.c
index 25d55dd63b407a6cc171b5e0b49e03bbb179854b..2ccb3a12c7d4833efe007afc34f9f26fdd1e057b 100644 (file)
--- a/libbb/run_shell.c
+++ b/libbb/run_shell.c
@@ -36,18 +36,14 @@
 #if ENABLE_SELINUX
 static security_context_t current_sid;
 
-void
-renew_current_security_context(void)
+void FAST_FUNC renew_current_security_context(void)
 {
-       if (current_sid)
-               freecon(current_sid);  /* Release old context  */
+       freecon(current_sid);  /* Release old context  */
        getcon(&current_sid);  /* update */
 }
-void
-set_current_security_context(security_context_t sid)
+void FAST_FUNC set_current_security_context(security_context_t sid)
 {
-       if (current_sid)
-               freecon(current_sid);  /* Release old context  */
+       freecon(current_sid);  /* Release old context  */
        current_sid = sid;
 }
 
@@ -58,7 +54,7 @@ set_current_security_context(security_context_t sid)
    If ADDITIONAL_ARGS is nonzero, pass it to the shell as more
    arguments.  */
 
-void run_shell(const char *shell, int loginshell, const char *command, const char **additional_args)
+void FAST_FUNC run_shell(const char *shell, int loginshell, const char *command, const char **additional_args)
 {
        const char **args;
        int argno = 1;
@@ -69,7 +65,7 @@ void run_shell(const char *shell, int loginshell, const char *command, const cha
 
        args = xmalloc(sizeof(char*) * (4 + additional_args_cnt));
 
-       args[0] = bb_get_last_path_component(xstrdup(shell));
+       args[0] = bb_get_last_path_component_nostrip(xstrdup(shell));
 
        if (loginshell)
                args[0] = xasprintf("-%s", args[0]);
@@ -84,10 +80,10 @@ void run_shell(const char *shell, int loginshell, const char *command, const cha
        }
        args[argno] = NULL;
 #if ENABLE_SELINUX
-       if (current_sid && !setexeccon(current_sid)) {
+       if (current_sid)
+               setexeccon(current_sid);
+       if (ENABLE_FEATURE_CLEAN_UP)
                freecon(current_sid);
-               execve(shell, (char **) args, environ);
-       } else
 #endif
        execv(shell, (char **) args);
        bb_perror_msg_and_die("cannot run %s", shell);