1 files changed, 32 insertions, 11 deletions

diff --git a/shell/hush.c b/shell/hush.c
index 1d2826d9a..46bb7e9a2 100644
--- a/shell/hush.c
+++ b/shell/hush.c
@@ -2185,7 +2185,7 @@ static NOINLINE int expand_vars_to_list(o_string *output, int n, char *arg, char
 
         ored_ch = 0;
 
-        debug_printf_expand("expand_vars_to_list: arg '%s'\n", arg);
+        debug_printf_expand("expand_vars_to_list: arg:'%s' or_mask:%x\n", arg, or_mask);
         debug_print_list("expand_vars_to_list", output, n);
         n = o_save_ptr(output, n);
         debug_print_list("expand_vars_to_list[0]", output, n);
@@ -3425,7 +3425,7 @@ static void pseudo_exec_argv(nommu_save_t *nommu_save,
         sigprocmask(SIG_SETMASK, &G.inherited_set, NULL);
         execvp(argv[0], argv);
         bb_perror_msg("can't execute '%s'", argv[0]);
-        _exit(EXIT_FAILURE);
+        _exit(127); /* bash compat */
 }
 
 /* Called after [v]fork() in run_pipe
@@ -3895,7 +3895,7 @@ static NOINLINE int run_pipe(struct pipe *pi)
                         argv_expanded = expand_strvec_to_strvec(argv + command->assignment_cnt);
                 }
 
-                /* if someone gives us an empty string: ``, $(), ... */
+                /* if someone gives us an empty string: `cmd with empty output` */
                 if (!argv_expanded[0]) {
                         debug_leave();
                         return 0;
@@ -5802,7 +5802,7 @@ static int parse_stream_dquoted(o_string *as_string,
         if (ch != '\n') {
                 next = i_peek(input);
         }
-        debug_printf_parse(": ch=%c (%d) escape=%d\n",
+        debug_printf_parse("\" ch=%c (%d) escape=%d\n",
                                         ch, ch, dest->o_escape);
         if (ch == '\\') {
                 if (next == EOF) {
@@ -5882,6 +5882,11 @@ static struct pipe *parse_stream(char **pstring,
                         end_trigger ? end_trigger : 'X');
         debug_enter();
 
+        /* If very first arg is "" or '', dest.data may end up NULL.
+         * Preventing this: */
+        o_addchr(&dest, '\0');
+        dest.length = 0;
+
         G.ifs = get_local_var_value("IFS");
         if (G.ifs == NULL)
                 G.ifs = " \t\n";
@@ -6979,16 +6984,32 @@ static int FAST_FUNC builtin_cd(char **argv)
 
 static int FAST_FUNC builtin_exec(char **argv)
 {
-        if (*++argv == NULL)
-                return EXIT_SUCCESS; /* bash does this */
-        {
+        static const char pseudo_null_str[] = { SPECIAL_VAR_SYMBOL, SPECIAL_VAR_SYMBOL, '\0' };
+        char **pp = argv;
 #if !BB_MMU
-                nommu_save_t dummy;
+        nommu_save_t dummy;
 #endif
-                /* TODO: if exec fails, bash does NOT exit! We do... */
-                pseudo_exec_argv(&dummy, argv, 0, NULL);
-                /* never returns */
+
+        if (*++argv == NULL)
+                return EXIT_SUCCESS; /* bash does this */
+
+        /* Make sure empty arguments aren't ignored */
+        /* Example: exec ls '' */
+        pp = argv;
+        while (*pp) {
+                if ((*pp)[0] == '\0')
+                        *pp = (char*)pseudo_null_str;
+                pp++;
         }
+
+        /* Careful: we can end up here after [v]fork. Do not restore
+         * tty pgrp then, only top-level shell process does that */
+        if (G_saved_tty_pgrp && getpid() == G.root_pid)
+                tcsetpgrp(G_interactive_fd, G_saved_tty_pgrp);
+
+        /* TODO: if exec fails, bash does NOT exit! We do... */
+        pseudo_exec_argv(&dummy, argv, 0, NULL);
+        /* never returns */
 }
 
 static int FAST_FUNC builtin_exit(char **argv)