hush: fix for nested $()s with escapes + testsuite
[oweals/busybox.git] / shell / hush.c
index aa740f1b45e42fe0f45fe53c74f3b1a402e428e5..77a3051913c9fc9973f01edf5fd61b6344b66f41 100644 (file)
@@ -20,7 +20,7 @@
  *      rewrites.
  *
  * Other credits:
- *      b_addchr() derived from similar w_addchar function in glibc-2.2
+ *      o_addchr() derived from similar w_addchar function in glibc-2.2
  *      setup_redirect(), redirect_opt_num(), and big chunks of main()
  *      and many builtins derived from contributions by Erik Andersen
  *      miscellaneous bugfixes from Matt Kraai
@@ -293,10 +293,6 @@ struct child_prog {
        smallint is_stopped;        /* is the program currently running? */
        struct redir_struct *redirects; /* I/O redirections */
        struct pipe *family;        /* pointer back to the child's parent pipe */
-       //sp counting seems to be broken... so commented out, grep for '//sp:'
-       //sp: int sp;               /* number of SPECIAL_VAR_SYMBOL */
-       //seems to be unused, grep for '//pt:'
-       //pt: int parse_type;
 };
 /* argv vector may contain variable references (^Cvar^C, ^C0^C etc)
  * and on execution these are substituted with their values.
@@ -341,8 +337,9 @@ typedef struct {
        int maxlen;
        smallint o_quote;
        smallint nonnull;
+       smallint has_empty_slot;
 } o_string;
-#define NULL_O_STRING {NULL,0,0,0,0}
+#define NULL_O_STRING { NULL }
 /* used for initialization: o_string foo = NULL_O_STRING; */
 
 /* I can almost use ordinary FILE *.  Is open_memstream() universally
@@ -360,8 +357,8 @@ struct in_str {
        int (*get) (struct in_str *);
        int (*peek) (struct in_str *);
 };
-#define b_getch(input) ((input)->get(input))
-#define b_peek(input) ((input)->peek(input))
+#define i_getch(input) ((input)->get(input))
+#define i_peek(input) ((input)->peek(input))
 
 enum {
        CHAR_ORDINARY           = 0,
@@ -464,8 +461,6 @@ enum { run_list_level = 0 };
 } while (0)
 
 
-#define B_CHUNK  100
-#define B_NOSPAC 1
 #define JOB_STATUS_FORMAT "[%d] %-22s %.40s\n"
 
 #if 1
@@ -494,11 +489,6 @@ static void syntax_lineno(int line)
 #endif
 
 /* Index of subroutines: */
-/*   o_string manipulation: */
-static int b_check_space(o_string *o, int len);
-static int b_addchr(o_string *o, int ch);
-static void b_reset(o_string *o);
-static int b_addqchr(o_string *o, int ch, int quote);
 /*  in_str manipulations: */
 static int static_get(struct in_str *i);
 static int static_peek(struct in_str *i);
@@ -516,8 +506,12 @@ static int free_pipe(struct pipe *pi, int indent);
 /*  really run the final data structures: */
 static int setup_redirects(struct child_prog *prog, int squirrel[]);
 static int run_list(struct pipe *pi);
-static void pseudo_exec_argv(char **argv) ATTRIBUTE_NORETURN;
-static void pseudo_exec(struct child_prog *child) ATTRIBUTE_NORETURN;
+#if BB_MMU
+#define pseudo_exec_argv(ptrs2free, argv)  pseudo_exec_argv(argv)
+#define      pseudo_exec(ptrs2free, child)      pseudo_exec(child)
+#endif
+static void pseudo_exec_argv(char **ptrs2free, char **argv) ATTRIBUTE_NORETURN;
+static void pseudo_exec(char **ptrs2free, struct child_prog *child) ATTRIBUTE_NORETURN;
 static int run_pipe(struct pipe *pi);
 /*   extended glob support: */
 static char **globhack(const char *src, char **strings);
@@ -535,12 +529,12 @@ static int done_pipe(struct p_context *ctx, pipe_style type);
 static int redirect_dup_num(struct in_str *input);
 static int redirect_opt_num(o_string *o);
 #if ENABLE_HUSH_TICK
-static int process_command_subs(o_string *dest, /*struct p_context *ctx,*/
+static int process_command_subs(o_string *dest,
                struct in_str *input, const char *subst_end);
 #endif
 static int parse_group(o_string *dest, struct p_context *ctx, struct in_str *input, int ch);
 static const char *lookup_param(const char *src);
-static int handle_dollar(o_string *dest, /*struct p_context *ctx,*/
+static int handle_dollar(o_string *dest,
                struct in_str *input);
 static int parse_stream(o_string *dest, struct p_context *ctx, struct in_str *input0, const char *end_trigger);
 /*   setup: */
@@ -619,6 +613,18 @@ static void free_strings(char **strings)
 }
 
 
+#if !BB_MMU
+#define EXTRA_PTRS 5 /* 1 for NULL, 1 for args, 3 for paranoid reasons */
+static char **alloc_ptrs(char **argv)
+{
+       char **v = argv;
+       while (*v)
+               v++;
+       return xzalloc((v - argv + EXTRA_PTRS) * sizeof(v[0]));
+}
+#endif
+
+
 /* Function prototypes for builtins */
 static int builtin_cd(char **argv);
 static int builtin_echo(char **argv);
@@ -698,9 +704,18 @@ static const struct built_in_command bltins[] = {
        BLTIN(NULL, NULL, NULL)
 };
 
+/* Signals are grouped, we handle them in batches */
+static void set_misc_sighandler(void (*handler)(int))
+{
+       bb_signals(0
+               + (1 << SIGINT)
+               + (1 << SIGQUIT)
+               + (1 << SIGTERM)
+               , handler);
+}
+
 #if ENABLE_HUSH_JOB
 
-/* Signals are grouped, we handle them in batches */
 static void set_fatal_sighandler(void (*handler)(int))
 {
        bb_signals(0
@@ -724,14 +739,6 @@ static void set_jobctrl_sighandler(void (*handler)(int))
                + (1 << SIGTTOU)
                , handler);
 }
-static void set_misc_sighandler(void (*handler)(int))
-{
-       bb_signals(0
-               + (1 << SIGINT)
-               + (1 << SIGQUIT)
-               + (1 << SIGTERM)
-               , handler);
-}
 /* SIGCHLD is special and handled separately */
 
 static void set_every_sighandler(void (*handler)(int))
@@ -814,7 +821,6 @@ static void hush_exit(int exitcode)
 
 #define set_fatal_sighandler(handler)   ((void)0)
 #define set_jobctrl_sighandler(handler) ((void)0)
-#define set_misc_sighandler(handler)    ((void)0)
 #define hush_exit(e)                    exit(e)
 
 #endif /* JOB */
@@ -891,9 +897,14 @@ static int builtin_exec(char **argv)
 {
        if (argv[1] == NULL)
                return EXIT_SUCCESS; /* bash does this */
+       {
+#if !BB_MMU
+               char **ptrs2free = alloc_ptrs(argv);
+#endif
 // FIXME: if exec fails, bash does NOT exit! We do...
-       pseudo_exec_argv(argv + 1);
-       /* never returns */
+               pseudo_exec_argv(ptrs2free, argv + 1);
+               /* never returns */
+       }
 }
 
 /* built-in 'exit' handler */
@@ -1155,57 +1166,167 @@ static int builtin_unset(char **argv)
 //     return EXIT_FAILURE;
 //}
 
-static int b_check_space(o_string *o, int len)
+/*
+ * o_string support
+ */
+#define B_CHUNK  (32 * sizeof(char*))
+
+static void o_reset(o_string *o)
+{
+       o->length = 0;
+       o->nonnull = 0;
+       if (o->data)
+               o->data[0] = '\0';
+}
+
+static void o_free(o_string *o)
+{
+       free(o->data);
+       memset(o, 0, sizeof(*o));
+}
+
+static void o_grow_by(o_string *o, int len)
 {
-       /* It would be easy to drop a more restrictive policy
-        * in here, such as setting a maximum string length */
        if (o->length + len > o->maxlen) {
-               /* assert(data == NULL || o->maxlen != 0); */
                o->maxlen += (2*len > B_CHUNK ? 2*len : B_CHUNK);
                o->data = xrealloc(o->data, 1 + o->maxlen);
        }
-       return o->data == NULL;
 }
 
-static int b_addchr(o_string *o, int ch)
+static void o_addchr(o_string *o, int ch)
 {
-       debug_printf("b_addchr: '%c' o->length=%d o=%p\n", ch, o->length, o);
-       if (b_check_space(o, 1))
-               return B_NOSPAC;
+       debug_printf("o_addchr: '%c' o->length=%d o=%p\n", ch, o->length, o);
+       o_grow_by(o, 1);
        o->data[o->length] = ch;
        o->length++;
        o->data[o->length] = '\0';
-       return 0;
-}
-
-static void b_reset(o_string *o)
-{
-       o->length = 0;
-       o->nonnull = 0;
-       if (o->data)
-               o->data[0] = '\0';
 }
 
-static void b_free(o_string *o)
+static void o_addstr(o_string *o, const char *str, int len)
 {
-       free(o->data);
-       memset(o, 0, sizeof(*o));
+       o_grow_by(o, len);
+       memcpy(&o->data[o->length], str, len);
+       o->length += len;
+       o->data[o->length] = '\0';
 }
 
 /* My analysis of quoting semantics tells me that state information
  * is associated with a destination, not a source.
  */
-static int b_addqchr(o_string *o, int ch, int quote)
+static void o_addqchr(o_string *o, int ch, int quote)
 {
        if (quote && strchr("*?[\\", ch)) {
-               int rc;
-               rc = b_addchr(o, '\\');
-               if (rc)
-                       return rc;
+               o_addchr(o, '\\');
+       }
+       o_addchr(o, ch);
+}
+
+static void o_addqstr(o_string *o, const char *str, int len, int quote)
+{
+       char ch;
+       if (!quote || str[strcspn(str, "*?[\\")] == '\0') {
+               o_addstr(o, str, len);
+               return;
+       }
+       while (len) {
+               ch = *str++;
+               if (ch && strchr("*?[\\", ch)) {
+                       o_addchr(o, '\\');
+               }
+               o_addchr(o, ch);
+               len--;
+       }
+}
+
+/* A special kind of o_string for $VAR and `cmd` expansion.
+ * It contains char* list[] at the beginning, which is grown in 16 element
+ * increments. Actual string data starts at the next multiple of 16.
+ * list[i] contains an INDEX (int!) into this string data.
+ * It means that if list[] needs to grow, data needs to be moved higher up
+ * but list[i]'s need not be modified.
+ * NB: remembering how many list[i]'s you have there is crucial.
+ * o_finalize_list() operation post-processes this structure - calculates
+ * and stores actual char* ptrs in list[]. Oh, it NULL terminates it as well.
+ */
+static int o_save_ptr(o_string *o, int n)
+{
+       char **list = (char**)o->data;
+       int string_start;
+       int string_len;
+
+       if (!o->has_empty_slot) {
+               string_start = ((n + 0xf) & ~0xf) * sizeof(list[0]);
+               string_len = o->length - string_start;
+               if (!(n & 0xf)) { /* 0, 0x10, 0x20...? */
+                       //bb_error_msg("list[%d]=%d string_start=%d (growing)", n, string_len, string_start);
+                       /* list[n] points to string_start, make space for 16 more pointers */
+                       o->maxlen += 0x10 * sizeof(list[0]);
+                       o->data = xrealloc(o->data, o->maxlen + 1);
+                       list = (char**)o->data;
+                       memmove(list + n + 0x10, list + n, string_len);
+                       o->length += 0x10 * sizeof(list[0]);
+               }
+               //else bb_error_msg("list[%d]=%d string_start=%d", n, string_len, string_start);
+       } else {
+               /* We have empty slot at list[n], reuse without growth */
+               string_start = ((n+1 + 0xf) & ~0xf) * sizeof(list[0]); /* NB: n+1! */
+               string_len = o->length - string_start;
+               //bb_error_msg("list[%d]=%d string_start=%d (empty slot)", n, string_len, string_start);
+               o->has_empty_slot = 0;
+       }
+       list[n] = (char*)string_len;
+       return n + 1;
+}
+
+static int o_get_last_ptr(o_string *o, int n)
+{
+       char **list = (char**)o->data;
+       int string_start = ((n + 0xf) & ~0xf) * sizeof(list[0]);
+
+       return ((int)list[n-1]) + string_start;
+}
+
+static char **o_finalize_list(o_string *o, int n)
+{
+       char **list = (char**)o->data;
+       int string_start;
+
+       o_save_ptr(o, n); /* force growth for list[n] if necessary */
+       string_start = ((n+1 + 0xf) & ~0xf) * sizeof(list[0]);
+       list[n] = NULL;
+       while (n) {
+               n--;
+               list[n] = o->data + (int)list[n] + string_start;
+       }
+       return list;
+}
+
+#ifdef DEBUG_EXPAND
+static void o_debug_list(const char *prefix, o_string *o, int n)
+{
+       char **list = (char**)o->data;
+       int string_start = ((n + 0xf) & ~0xf) * sizeof(list[0]);
+       int i = 0;
+       fprintf(stderr, "%s: list:%p n:%d string_start:%d length:%d maxlen:%d\n",
+                       prefix, list, n, string_start, o->length, o->maxlen);
+       while (i < n) {
+               fprintf(stderr, " list[%d]=%d '%s'\n", i, (int)list[i],
+                               o->data + (int)list[i] + string_start);
+               i++;
+       }
+       if (n) {
+               const char *p = o->data + (int)list[n] + string_start;
+               fprintf(stderr, " total_sz:%d\n", (p + strlen(p) + 1) - o->data);
        }
-       return b_addchr(o, ch);
 }
+#else
+#define o_debug_list(prefix, o, n) ((void)0)
+#endif
 
+
+/*
+ * in_str support
+ */
 static int static_get(struct in_str *i)
 {
        int ch = *i->p++;
@@ -1421,7 +1542,7 @@ static void restore_redirects(int squirrel[])
  * XXX no exit() here.  If you don't exec, use _exit instead.
  * The at_exit handlers apparently confuse the calling process,
  * in particular stdin handling.  Not sure why? -- because of vfork! (vda) */
-static void pseudo_exec_argv(char **argv)
+static void pseudo_exec_argv(char **ptrs2free, char **argv)
 {
        int i, rcode;
        char *p;
@@ -1430,8 +1551,10 @@ static void pseudo_exec_argv(char **argv)
        for (i = 0; is_assignment(argv[i]); i++) {
                debug_printf_exec("pid %d environment modification: %s\n",
                                getpid(), argv[i]);
-// FIXME: vfork case??
                p = expand_string_to_string(argv[i]);
+#if !BB_MMU
+               *ptrs2free++ = p;
+#endif
                putenv(p);
        }
        argv += i;
@@ -1442,6 +1565,9 @@ static void pseudo_exec_argv(char **argv)
                _exit(EXIT_SUCCESS);
 
        argv = expand_strvec_to_strvec(argv);
+#if !BB_MMU
+       *ptrs2free++ = (char*) argv;
+#endif
 
        /*
         * Check if the command matches any of the builtins.
@@ -1480,18 +1606,18 @@ static void pseudo_exec_argv(char **argv)
        debug_printf_exec("execing '%s'\n", argv[0]);
        execvp(argv[0], argv);
        bb_perror_msg("cannot exec '%s'", argv[0]);
-       _exit(1);
+       _exit(EXIT_FAILURE);
 }
 
 /* Called after [v]fork() in run_pipe()
  */
-static void pseudo_exec(struct child_prog *child)
+static void pseudo_exec(char **ptrs2free, struct child_prog *child)
 {
 // FIXME: buggy wrt NOMMU! Must not modify any global data
 // until it does exec/_exit, but currently it does
 // (puts malloc'ed stuff into environment)
        if (child->argv)
-               pseudo_exec_argv(child->argv);
+               pseudo_exec_argv(ptrs2free, child->argv);
 
        if (child->group) {
 #if !BB_MMU
@@ -1825,7 +1951,6 @@ static int run_pipe(struct pipe *pi)
                }
                for (i = 0; is_assignment(argv[i]); i++) {
                        p = expand_string_to_string(argv[i]);
-                       //sp: child->sp--;
                        putenv(p);
                }
                for (x = bltins; x->cmd; x++) {
@@ -1842,7 +1967,6 @@ static int run_pipe(struct pipe *pi)
                                 * things seem to work with glibc. */
                                setup_redirects(child, squirrel);
                                debug_printf_exec(": builtin '%s' '%s'...\n", x->cmd, argv[i+1]);
-                               //sp: if (child->sp) /* btw we can do it unconditionally... */
                                argv_expanded = expand_strvec_to_strvec(argv + i);
                                rcode = x->function(argv_expanded) & 0xff;
                                free(argv_expanded);
@@ -1858,7 +1982,6 @@ static int run_pipe(struct pipe *pi)
                                setup_redirects(child, squirrel);
                                save_nofork_data(&nofork_save);
                                argv_expanded = argv + i;
-                               //sp: if (child->sp)
                                argv_expanded = expand_strvec_to_strvec(argv + i);
                                debug_printf_exec(": run_nofork_applet '%s' '%s'...\n", argv_expanded[0], argv_expanded[1]);
                                rcode = run_nofork_applet_prime(&nofork_save, a, argv_expanded);
@@ -1880,10 +2003,16 @@ static int run_pipe(struct pipe *pi)
        nextin = 0;
 
        for (i = 0; i < pi->num_progs; i++) {
+#if !BB_MMU
+               char **ptrs2free = NULL;
+#endif
                child = &(pi->progs[i]);
-               if (child->argv)
+               if (child->argv) {
                        debug_printf_exec(": pipe member '%s' '%s'...\n", child->argv[0], child->argv[1]);
-               else
+#if !BB_MMU
+                       ptrs2free = alloc_ptrs(child->argv);
+#endif
+               } else
                        debug_printf_exec(": pipe member with no argv\n");
 
                /* pipes are inserted between pairs of commands */
@@ -1925,9 +2054,11 @@ static int run_pipe(struct pipe *pi)
                        set_jobctrl_sighandler(SIG_DFL);
                        set_misc_sighandler(SIG_DFL);
                        signal(SIGCHLD, SIG_DFL);
-                       pseudo_exec(child); /* does not return */
+                       pseudo_exec(ptrs2free, child); /* does not return */
                }
-
+#if !BB_MMU
+               free_strings(ptrs2free);
+#endif
                if (child->pid < 0) { /* [v]fork failed */
                        /* Clearly indicate, was it fork or vfork */
                        bb_perror_msg(BB_MMU ? "fork" : "vfork");
@@ -2331,7 +2462,7 @@ static int run_and_free_list(struct pipe *pi)
         * In the long run that function can be merged with run_list,
         * but doing that now would hobble the debugging effort. */
        free_pipe_list(pi, /* indent: */ 0);
-       debug_printf_exec("run_nad_free_list return %d\n", rcode);
+       debug_printf_exec("run_and_free_list return %d\n", rcode);
        return rcode;
 }
 
@@ -2427,106 +2558,25 @@ static int xglob(o_string *dest, char ***pglob)
  * followed by strings themself.
  * Caller can deallocate entire list by single free(list). */
 
-/* Helpers first:
- * count_XXX estimates size of the block we need. It's okay
- * to over-estimate sizes a bit, if it makes code simpler */
-static int count_ifs(const char *str)
-{
-       int cnt = 0;
-       debug_printf_expand("count_ifs('%s') ifs='%s'", str, ifs);
-       while (1) {
-               str += strcspn(str, ifs);
-               if (!*str) break;
-               str++; /* str += strspn(str, ifs); */
-               cnt++; /* cnt += strspn(str, ifs); - but this code is larger */
-       }
-       debug_printf_expand(" return %d\n", cnt);
-       return cnt;
-}
-
-static void count_var_expansion_space(int *countp, int *lenp, char *arg)
-{
-       char first_ch;
-       int i;
-       int len = *lenp;
-       int count = *countp;
-       const char *val;
-       char *p;
-
-       while ((p = strchr(arg, SPECIAL_VAR_SYMBOL))) {
-               len += p - arg;
-               arg = ++p;
-               p = strchr(p, SPECIAL_VAR_SYMBOL);
-               first_ch = arg[0];
-
-               switch (first_ch & 0x7f) {
-               /* high bit in 1st_ch indicates that var is double-quoted */
-               case '$': /* pid */
-               case '!': /* bg pid */
-               case '?': /* exitcode */
-               case '#': /* argc */
-                       len += sizeof(int)*3 + 1; /* enough for int */
-                       break;
-               case '*':
-               case '@':
-                       for (i = 1; global_argv[i]; i++) {
-                               len += strlen(global_argv[i]) + 1;
-                               count++;
-                               if (!(first_ch & 0x80))
-                                       count += count_ifs(global_argv[i]);
-                       }
-                       break;
-               default:
-                       *p = '\0';
-                       arg[0] = first_ch & 0x7f;
-                       if (isdigit(arg[0])) {
-                               i = xatoi_u(arg);
-                               val = NULL;
-                               if (i < global_argc)
-                                       val = global_argv[i];
-                       } else
-                               val = lookup_param(arg);
-                       arg[0] = first_ch;
-                       *p = SPECIAL_VAR_SYMBOL;
-
-                       if (val) {
-                               len += strlen(val) + 1;
-                               if (!(first_ch & 0x80))
-                                       count += count_ifs(val);
-                       }
-               }
-               arg = ++p;
-       }
-
-       len += strlen(arg) + 1;
-       count++;
-       *lenp = len;
-       *countp = count;
-}
-
 /* Store given string, finalizing the word and starting new one whenever
  * we encounter ifs char(s). This is used for expanding variable values.
  * End-of-string does NOT finalize word: think about 'echo -$VAR-' */
-static int expand_on_ifs(char **list, int n, char **posp, const char *str)
+static int expand_on_ifs(o_string *output, int n, const char *str)
 {
-       char *pos = *posp;
        while (1) {
                int word_len = strcspn(str, ifs);
                if (word_len) {
-                       memcpy(pos, str, word_len); /* store non-ifs chars */
-                       pos += word_len;
+                       o_addqstr(output, str, word_len, output->o_quote);
                        str += word_len;
                }
                if (!*str)  /* EOL - do not finalize word */
                        break;
-               *pos++ = '\0';
-               if (n) debug_printf_expand("expand_on_ifs finalized list[%d]=%p '%s' "
-                       "strlen=%d next=%p pos=%p\n", n-1, list[n-1], list[n-1],
-                       strlen(list[n-1]), list[n-1] + strlen(list[n-1]) + 1, pos);
-               list[n++] = pos;
+               o_addchr(output, '\0');
+               o_debug_list("expand_on_ifs", output, n);
+               n = o_save_ptr(output, n);
                str += strspn(str, ifs); /* skip ifs chars */
        }
-       *posp = pos;
+       o_debug_list("expand_on_ifs[1]", output, n);
        return n;
 }
 
@@ -2537,7 +2587,7 @@ static int expand_on_ifs(char **list, int n, char **posp, const char *str)
  * 'echo -$*-'. If you play here, you must run testsuite afterwards! */
 /* NB: another bug is that we cannot detect empty strings yet:
  * "" or $empty"" expands to zero words, has to expand to empty word */
-static int expand_vars_to_list(char **list, int n, char **posp, char *arg, char or_mask)
+static int expand_vars_to_list(o_string *output, int n, char *arg, char or_mask)
 {
        /* or_mask is either 0 (normal case) or 0x80
         * (expansion of right-hand side of assignment == 1-element expand) */
@@ -2546,18 +2596,19 @@ static int expand_vars_to_list(char **list, int n, char **posp, char *arg, char
        int i;
        const char *val;
        char *p;
-       char *pos = *posp;
 
        ored_ch = 0;
 
-       if (n) debug_printf_expand("expand_vars_to_list finalized list[%d]=%p '%s' "
-               "strlen=%d next=%p pos=%p\n", n-1, list[n-1], list[n-1],
-               strlen(list[n-1]), list[n-1] + strlen(list[n-1]) + 1, pos);
-       list[n++] = pos;
+       debug_printf_expand("expand_vars_to_list: arg '%s'\n", arg);
+       o_debug_list("expand_vars_to_list", output, n);
+       n = o_save_ptr(output, n);
+       o_debug_list("expand_vars_to_list[0]", output, n);
 
-       while ((p = strchr(arg, SPECIAL_VAR_SYMBOL))) {
-               memcpy(pos, arg, p - arg);
-               pos += (p - arg);
+       while ((p = strchr(arg, SPECIAL_VAR_SYMBOL)) != NULL) {
+               o_string subst_result = NULL_O_STRING;
+
+               o_addqstr(output, arg, p - arg, output->o_quote);
+               o_debug_list("expand_vars_to_list[1]", output, n);
                arg = ++p;
                p = strchr(p, SPECIAL_VAR_SYMBOL);
 
@@ -2586,16 +2637,15 @@ static int expand_vars_to_list(char **list, int n, char **posp, char *arg, char
                                break;
                        if (!(first_ch & 0x80)) { /* unquoted $* or $@ */
                                while (global_argv[i]) {
-                                       n = expand_on_ifs(list, n, &pos, global_argv[i]);
+                                       n = expand_on_ifs(output, n, global_argv[i]);
                                        debug_printf_expand("expand_vars_to_list: argv %d (last %d)\n", i, global_argc-1);
                                        if (global_argv[i++][0] && global_argv[i]) {
                                                /* this argv[] is not empty and not last:
                                                 * put terminating NUL, start new word */
-                                               *pos++ = '\0';
-                                               if (n) debug_printf_expand("expand_vars_to_list 2 finalized list[%d]=%p '%s' "
-                                                       "strlen=%d next=%p pos=%p\n", n-1, list[n-1], list[n-1],
-                                                       strlen(list[n-1]), list[n-1] + strlen(list[n-1]) + 1, pos);
-                                               list[n++] = pos;
+                                               o_addchr(output, '\0');
+                                               o_debug_list("expand_vars_to_list[2]", output, n);
+                                               n = o_save_ptr(output, n);
+                                               o_debug_list("expand_vars_to_list[3]", output, n);
                                        }
                                }
                        } else
@@ -2603,27 +2653,34 @@ static int expand_vars_to_list(char **list, int n, char **posp, char *arg, char
                         * and in this case should treat it like '$*' - see 'else...' below */
                        if (first_ch == ('@'|0x80) && !or_mask) { /* quoted $@ */
                                while (1) {
-                                       strcpy(pos, global_argv[i]);
-                                       pos += strlen(global_argv[i]);
+                                       o_addqstr(output, global_argv[i], strlen(global_argv[i]), output->o_quote);
                                        if (++i >= global_argc)
                                                break;
-                                       *pos++ = '\0';
-                                       if (n) debug_printf_expand("expand_vars_to_list 3 finalized list[%d]=%p '%s' "
-                                               "strlen=%d next=%p pos=%p\n", n-1, list[n-1], list[n-1],
-                                                       strlen(list[n-1]), list[n-1] + strlen(list[n-1]) + 1, pos);
-                                       list[n++] = pos;
+                                       o_addchr(output, '\0');
+                                       o_debug_list("expand_vars_to_list[4]", output, n);
+                                       n = o_save_ptr(output, n);
                                }
                        } else { /* quoted $*: add as one word */
                                while (1) {
-                                       strcpy(pos, global_argv[i]);
-                                       pos += strlen(global_argv[i]);
+                                       o_addqstr(output, global_argv[i], strlen(global_argv[i]), output->o_quote);
                                        if (!global_argv[++i])
                                                break;
                                        if (ifs[0])
-                                               *pos++ = ifs[0];
+                                               o_addchr(output, ifs[0]);
                                }
                        }
                        break;
+               case '`': {
+                       struct in_str input;
+                       *p = '\0';
+                       arg++;
+                       //bb_error_msg("SUBST '%s' first_ch %x", arg, first_ch);
+                       setup_string_in_str(&input, arg);
+                       process_command_subs(&subst_result, &input, NULL);
+                       //bb_error_msg("RES '%s'", subst_result.data);
+                       val = subst_result.data;
+                       goto store_val;
+               }
                default:
                        *p = '\0';
                        arg[0] = first_ch & 0x7f;
@@ -2635,66 +2692,54 @@ static int expand_vars_to_list(char **list, int n, char **posp, char *arg, char
                        } else
                                val = lookup_param(arg);
                        arg[0] = first_ch;
+ store_val:
                        *p = SPECIAL_VAR_SYMBOL;
                        if (!(first_ch & 0x80)) { /* unquoted $VAR */
                                if (val) {
-                                       n = expand_on_ifs(list, n, &pos, val);
+                                       n = expand_on_ifs(output, n, val);
                                        val = NULL;
                                }
-                       } /* else: quoted $VAR, val will be appended at pos */
+                       } /* else: quoted $VAR, val will be appended below */
                }
                if (val) {
-                       strcpy(pos, val);
-                       pos += strlen(val);
+                       o_addqstr(output, val, strlen(val), output->o_quote);
                }
+
+               o_free(&subst_result);
                arg = ++p;
+       } /* end of "while (SPECIAL_VAR_SYMBOL is found) ..." */
+
+       if (arg[0]) {
+               o_debug_list("expand_vars_to_list[a]", output, n);
+               o_addqstr(output, arg, strlen(arg) + 1, output->o_quote);
+               o_debug_list("expand_vars_to_list[b]", output, n);
+       } else if (output->length == o_get_last_ptr(output, n) /* expansion is empty */
+        && !(ored_ch & 0x80) /* and all vars were not quoted. */
+       ) {
+               n--;
+               /* allow to reuse list[n] later without re-growth */
+               output->has_empty_slot = 1;
+       } else {
+               o_addchr(output, '\0');
        }
-       debug_printf_expand("expand_vars_to_list adding tail '%s' at %p\n", arg, pos);
-       strcpy(pos, arg);
-       pos += strlen(arg) + 1;
-       if (pos == list[n-1] + 1) { /* expansion is empty */
-               if (!(ored_ch & 0x80)) { /* all vars were not quoted... */
-                       debug_printf_expand("expand_vars_to_list list[%d] empty, going back\n", n);
-                       pos--;
-                       n--;
-               }
-       }
-
-       *posp = pos;
        return n;
 }
 
 static char **expand_variables(char **argv, char or_mask)
 {
        int n;
-       int count = 1;
-       int len = 0;
-       char *pos, **v, **list;
+       char **list;
+       char **v;
+       o_string output = NULL_O_STRING;
 
-       v = argv;
-       if (!*v) debug_printf_expand("count_var_expansion_space: "
-                       "argv[0]=NULL count=%d len=%d alloc_space=%d\n",
-                       count, len, sizeof(char*) * count + len);
-       while (*v) {
-               count_var_expansion_space(&count, &len, *v);
-               debug_printf_expand("count_var_expansion_space: "
-                       "'%s' count=%d len=%d alloc_space=%d\n",
-                       *v, count, len, sizeof(char*) * count + len);
-               v++;
-       }
-       len += sizeof(char*) * count; /* total to alloc */
-       list = xmalloc(len);
-       pos = (char*)(list + count);
-       debug_printf_expand("list=%p, list[0] should be %p\n", list, pos);
        n = 0;
        v = argv;
        while (*v)
-               n = expand_vars_to_list(list, n, &pos, *v++, or_mask);
+               n = expand_vars_to_list(&output, n, *v++, or_mask);
+       o_debug_list("expand_variables", &output, n);
 
-       if (n) debug_printf_expand("finalized list[%d]=%p '%s' "
-               "strlen=%d next=%p pos=%p\n", n-1, list[n-1], list[n-1],
-               strlen(list[n-1]), list[n-1] + strlen(list[n-1]) + 1, pos);
-       list[n] = NULL;
+       /* output.data (malloced) gets returned in "list" */
+       list = o_finalize_list(&output, n);
 
 #ifdef DEBUG_EXPAND
        {
@@ -2703,12 +2748,8 @@ static char **expand_variables(char **argv, char or_mask)
                        debug_printf_expand("list[%d]=%p '%s'\n", m, list[m], list[m]);
                        m++;
                }
-               debug_printf_expand("used_space=%d\n", pos - (char*)list);
        }
 #endif
-       if (ENABLE_HUSH_DEBUG)
-               if (pos - (char*)list > len)
-                       bb_error_msg_and_die("BUG in varexp");
        return list;
 }
 
@@ -2999,7 +3040,7 @@ static int reserved_word(o_string *dest, struct p_context *ctx)
                        if (ctx->res_w == RES_IN || ctx->res_w == RES_FOR) {
                                syntax("malformed for"); /* example: 'for if' */
                                ctx->res_w = RES_SNTX;
-                               b_reset(dest);
+                               o_reset(dest);
                                return 1;
                        }
 #endif
@@ -3010,7 +3051,7 @@ static int reserved_word(o_string *dest, struct p_context *ctx)
                } else if (ctx->res_w == RES_NONE || !(ctx->old_flag & (1 << r->code))) {
                        syntax(NULL);
                        ctx->res_w = RES_SNTX;
-                       b_reset(dest);
+                       o_reset(dest);
                        return 1;
                }
                ctx->res_w = r->code;
@@ -3025,7 +3066,7 @@ static int reserved_word(o_string *dest, struct p_context *ctx)
                        *ctx = *old;   /* physical copy */
                        free(old);
                }
-               b_reset(dest);
+               o_reset(dest);
                return 1;
        }
        return 0;
@@ -3070,7 +3111,7 @@ static int done_word(o_string *dest, struct p_context *ctx)
                return 1;
        }
 
-       b_reset(dest);
+       o_reset(dest);
        if (ctx->pending_redirect) {
                /* NB: don't free_strings(ctx->pending_redirect->glob_word) here */
                if (ctx->pending_redirect->glob_word
@@ -3129,8 +3170,6 @@ static int done_command(struct p_context *ctx)
        /*child->is_stopped = 0;*/
        /*child->group = NULL;*/
        child->family = pi;
-       //sp: /*child->sp = 0;*/
-       //pt: child->parse_type = ctx->parse_type;
 
        ctx->child = child;
        /* but ctx->pipe and ctx->list_head remain unchanged */
@@ -3168,20 +3207,20 @@ static int done_pipe(struct p_context *ctx, pipe_style type)
 static int redirect_dup_num(struct in_str *input)
 {
        int ch, d = 0, ok = 0;
-       ch = b_peek(input);
+       ch = i_peek(input);
        if (ch != '&') return -1;
 
-       b_getch(input);  /* get the & */
-       ch = b_peek(input);
+       i_getch(input);  /* get the & */
+       ch = i_peek(input);
        if (ch == '-') {
-               b_getch(input);
+               i_getch(input);
                return -3;  /* "-" represents "close me" */
        }
        while (isdigit(ch)) {
                d = d*10 + (ch-'0');
                ok = 1;
-               b_getch(input);
-               ch = b_peek(input);
+               i_getch(input);
+               ch = i_peek(input);
        }
        if (ok) return d;
 
@@ -3213,12 +3252,11 @@ static int redirect_opt_num(o_string *o)
        }
        /* reuse num (and save an int) */
        num = atoi(o->data);
-       b_reset(o);
+       o_reset(o);
        return num;
 }
 
 #if ENABLE_HUSH_TICK
-/* NB: currently disabled on NOMMU */
 static FILE *generate_stream_from_list(struct pipe *head)
 {
        FILE *pf;
@@ -3229,6 +3267,10 @@ static FILE *generate_stream_from_list(struct pipe *head)
 /* By using vfork here, we suspend parent till child exits or execs.
  * If child will not do it before it fills the pipe, it can block forever
  * in write(STDOUT_FILENO), and parent (shell) will be also stuck.
+ * Try this script:
+ * yes "0123456789012345678901234567890" | dd bs=32 count=64k >TESTFILE
+ * huge=`cat TESTFILE` # will block here forever
+ * echo OK
  */
        pid = BB_MMU ? fork() : vfork();
        if (pid < 0)
@@ -3260,7 +3302,6 @@ static FILE *generate_stream_from_list(struct pipe *head)
 
 /* Return code is exit status of the process that is run. */
 static int process_command_subs(o_string *dest,
-               /*struct p_context *ctx,*/
                struct in_str *input,
                const char *subst_end)
 {
@@ -3278,7 +3319,7 @@ static int process_command_subs(o_string *dest,
                return retcode;  /* syntax error or EOF */
        done_word(&result, &inner);
        done_pipe(&inner, PIPE_SEQ);
-       b_free(&result);
+       o_free(&result);
 
        p = generate_stream_from_list(inner.list_head);
        if (p == NULL)
@@ -3288,16 +3329,16 @@ static int process_command_subs(o_string *dest,
 
        /* now send results of command back into original context */
        eol_cnt = 0;
-       while ((ch = b_getch(&pipe_str)) != EOF) {
+       while ((ch = i_getch(&pipe_str)) != EOF) {
                if (ch == '\n') {
                        eol_cnt++;
                        continue;
                }
                while (eol_cnt) {
-                       b_addqchr(dest, '\n', dest->o_quote);
+                       o_addqchr(dest, '\n', dest->o_quote);
                        eol_cnt--;
                }
-               b_addqchr(dest, ch, dest->o_quote);
+               o_addqchr(dest, ch, dest->o_quote);
        }
 
        debug_printf("done reading from pipe, pclose()ing\n");
@@ -3354,34 +3395,146 @@ static const char *lookup_param(const char *src)
        return NULL;
 }
 
+#if ENABLE_HUSH_TICK
+/* Subroutines for copying $(...) and `...` things */
+static void add_till_backquote(o_string *dest, struct in_str *input);
+/* '...' */
+static void add_till_single_quote(o_string *dest, struct in_str *input)
+{
+       while (1) {
+               int ch = i_getch(input);
+               if (ch == EOF)
+                       break;
+               if (ch == '\'')
+                       break;
+               o_addqchr(dest, ch, 1);
+       }
+}
+/* "...\"...`..`...." - do we need to handle "...$(..)..." too? */
+static void add_till_double_quote(o_string *dest, struct in_str *input)
+{
+       while (1) {
+               int ch = i_getch(input);
+               if (ch == '"')
+                       break;
+               if (ch == '\\') {  /* \x. Copy both chars. */
+                       o_addqchr(dest, ch, 1);
+                       ch = i_getch(input);
+               }
+               if (ch == EOF)
+                       break;
+               o_addqchr(dest, ch, 1);
+               if (ch == '`') {
+                       add_till_backquote(dest, input);
+                       o_addqchr(dest, ch, 1);
+                       continue;
+               }
+//             if (ch == '$') ...
+       }
+}
+/* Process `cmd` - copy contents until "`" is seen. Complicated by
+ * \` quoting.
+ * "Within the backquoted style of command substitution, backslash
+ * shall retain its literal meaning, except when followed by: '$', '`', or '\'.
+ * The search for the matching backquote shall be satisfied by the first
+ * backquote found without a preceding backslash; during this search,
+ * if a non-escaped backquote is encountered within a shell comment,
+ * a here-document, an embedded command substitution of the $(command)
+ * form, or a quoted string, undefined results occur. A single-quoted
+ * or double-quoted string that begins, but does not end, within the
+ * "`...`" sequence produces undefined results."
+ * Example                               Output
+ * echo `echo '\'TEST\`echo ZZ\`BEST`    \TESTZZBEST
+ */
+static void add_till_backquote(o_string *dest, struct in_str *input)
+{
+       while (1) {
+               int ch = i_getch(input);
+               if (ch == '`')
+                       break;
+               if (ch == '\\') {  /* \x. Copy both chars unless it is \` */
+                       int ch2 = i_getch(input);
+                       if (ch2 != '`' && ch2 != '$' && ch2 != '\\')
+                               o_addqchr(dest, ch, 1);
+                       ch = ch2;
+               }
+               if (ch == EOF)
+                       break;
+               o_addqchr(dest, ch, 1);
+       }
+}
+/* Process $(cmd) - copy contents until ")" is seen. Complicated by
+ * quoting and nested ()s.
+ * "With the $(command) style of command substitution, all characters
+ * following the open parenthesis to the matching closing parenthesis
+ * constitute the command. Any valid shell script can be used for command,
+ * except a script consisting solely of redirections which produces
+ * unspecified results."
+ * Example                              Output
+ * echo $(echo '(TEST)' BEST)           (TEST) BEST
+ * echo $(echo 'TEST)' BEST)            TEST) BEST
+ * echo $(echo \(\(TEST\) BEST)         ((TEST) BEST
+ */
+static void add_till_closing_curly_brace(o_string *dest, struct in_str *input)
+{
+       int count = 0;
+       while (1) {
+               int ch = i_getch(input);
+               if (ch == EOF)
+                       break;
+               if (ch == '(')
+                       count++;
+               if (ch == ')')
+                       if (--count < 0)
+                               break;
+               o_addqchr(dest, ch, 1);
+               if (ch == '\'') {
+                       add_till_single_quote(dest, input);
+                       o_addqchr(dest, ch, 1);
+                       continue;
+               }
+               if (ch == '"') {
+                       add_till_double_quote(dest, input);
+                       o_addqchr(dest, ch, 1);
+                       continue;
+               }
+               if (ch == '\\') { /* \x. Copy verbatim. Important for  \(, \) */
+                       ch = i_getch(input);
+                       if (ch == EOF)
+                               break;
+                       o_addqchr(dest, ch, 1);
+                       continue;
+               }
+       }
+}
+#endif /* ENABLE_HUSH_TICK */
+
 /* return code: 0 for OK, 1 for syntax error */
-static int handle_dollar(o_string *dest, /*struct p_context *ctx,*/ struct in_str *input)
+static int handle_dollar(o_string *dest, struct in_str *input)
 {
-       int ch = b_peek(input);  /* first character after the $ */
+       int ch = i_peek(input);  /* first character after the $ */
        unsigned char quote_mask = dest->o_quote ? 0x80 : 0;
 
        debug_printf_parse("handle_dollar entered: ch='%c'\n", ch);
        if (isalpha(ch)) {
-               b_addchr(dest, SPECIAL_VAR_SYMBOL);
-               //sp: ctx->child->sp++;
+               o_addchr(dest, SPECIAL_VAR_SYMBOL);
                while (1) {
                        debug_printf_parse(": '%c'\n", ch);
-                       b_getch(input);
-                       b_addchr(dest, ch | quote_mask);
+                       i_getch(input);
+                       o_addchr(dest, ch | quote_mask);
                        quote_mask = 0;
-                       ch = b_peek(input);
+                       ch = i_peek(input);
                        if (!isalnum(ch) && ch != '_')
                                break;
                }
-               b_addchr(dest, SPECIAL_VAR_SYMBOL);
+               o_addchr(dest, SPECIAL_VAR_SYMBOL);
        } else if (isdigit(ch)) {
  make_one_char_var:
-               b_addchr(dest, SPECIAL_VAR_SYMBOL);
-               //sp: ctx->child->sp++;
+               o_addchr(dest, SPECIAL_VAR_SYMBOL);
                debug_printf_parse(": '%c'\n", ch);
-               b_getch(input);
-               b_addchr(dest, ch | quote_mask);
-               b_addchr(dest, SPECIAL_VAR_SYMBOL);
+               i_getch(input);
+               o_addchr(dest, ch | quote_mask);
+               o_addchr(dest, SPECIAL_VAR_SYMBOL);
        } else switch (ch) {
                case '$': /* pid */
                case '!': /* last bg pid */
@@ -3391,12 +3544,11 @@ static int handle_dollar(o_string *dest, /*struct p_context *ctx,*/ struct in_st
                case '@': /* args */
                        goto make_one_char_var;
                case '{':
-                       b_addchr(dest, SPECIAL_VAR_SYMBOL);
-                       //sp: ctx->child->sp++;
-                       b_getch(input);
+                       o_addchr(dest, SPECIAL_VAR_SYMBOL);
+                       i_getch(input);
                        /* XXX maybe someone will try to escape the '}' */
                        while (1) {
-                               ch = b_getch(input);
+                               ch = i_getch(input);
                                if (ch == '}')
                                        break;
                                if (!isalnum(ch) && ch != '_') {
@@ -3405,16 +3557,22 @@ static int handle_dollar(o_string *dest, /*struct p_context *ctx,*/ struct in_st
                                        return 1;
                                }
                                debug_printf_parse(": '%c'\n", ch);
-                               b_addchr(dest, ch | quote_mask);
+                               o_addchr(dest, ch | quote_mask);
                                quote_mask = 0;
                        }
-                       b_addchr(dest, SPECIAL_VAR_SYMBOL);
+                       o_addchr(dest, SPECIAL_VAR_SYMBOL);
                        break;
 #if ENABLE_HUSH_TICK
-               case '(':
-                       b_getch(input);
-                       process_command_subs(dest, /*ctx,*/ input, ")");
+               case '(': {
+                       //int pos = dest->length;
+                       i_getch(input);
+                       o_addchr(dest, SPECIAL_VAR_SYMBOL);
+                       o_addchr(dest, quote_mask | '`');
+                       add_till_closing_curly_brace(dest, input);
+                       //bb_error_msg("RES '%s'", dest->data + pos);
+                       o_addchr(dest, SPECIAL_VAR_SYMBOL);
                        break;
+               }
 #endif
                case '-':
                case '_':
@@ -3423,7 +3581,7 @@ static int handle_dollar(o_string *dest, /*struct p_context *ctx,*/ struct in_st
                        return 1;
                        break;
                default:
-                       b_addqchr(dest, '$', dest->o_quote);
+                       o_addqchr(dest, '$', dest->o_quote);
        }
        debug_printf_parse("handle_dollar return 0\n");
        return 0;
@@ -3447,11 +3605,11 @@ static int parse_stream(o_string *dest, struct p_context *ctx,
        while (1) {
                m = CHAR_IFS;
                next = '\0';
-               ch = b_getch(input);
+               ch = i_getch(input);
                if (ch != EOF) {
                        m = charmap[ch];
                        if (ch != '\n')
-                               next = b_peek(input);
+                               next = i_peek(input);
                }
                debug_printf_parse(": ch=%c (%d) m=%d quote=%d\n",
                                                ch, ch, m, dest->o_quote);
@@ -3463,7 +3621,7 @@ static int parse_stream(o_string *dest, struct p_context *ctx,
                                debug_printf_parse("parse_stream return 1: unterminated \"\n");
                                return 1;
                        }
-                       b_addqchr(dest, ch, dest->o_quote);
+                       o_addqchr(dest, ch, dest->o_quote);
                        continue;
                }
                if (m == CHAR_IFS) {
@@ -3492,13 +3650,13 @@ static int parse_stream(o_string *dest, struct p_context *ctx,
                case '#':
                        if (dest->length == 0 && !dest->o_quote) {
                                while (1) {
-                                       ch = b_peek(input);
+                                       ch = i_peek(input);
                                        if (ch == EOF || ch == '\n')
                                                break;
-                                       b_getch(input);
+                                       i_getch(input);
                                }
                        } else {
-                               b_addqchr(dest, ch, dest->o_quote);
+                               o_addqchr(dest, ch, dest->o_quote);
                        }
                        break;
                case '\\':
@@ -3507,11 +3665,28 @@ static int parse_stream(o_string *dest, struct p_context *ctx,
                                debug_printf_parse("parse_stream return 1: \\<eof>\n");
                                return 1;
                        }
-                       b_addqchr(dest, '\\', dest->o_quote);
-                       b_addqchr(dest, b_getch(input), dest->o_quote);
+                       /* bash:
+                        * "The backslash retains its special meaning [in "..."]
+                        * only when followed by one of the following characters:
+                        * $, `, ", \, or <newline>.  A double quote may be quoted
+                        * within double quotes by preceding it with a  backslash.
+                        * If enabled, history expansion will be performed unless
+                        * an ! appearing in double quotes is escaped using
+                        * a backslash. The backslash preceding the ! is not removed."
+                        */
+                       if (dest->o_quote) {
+                               if (strchr("$`\"\\", next) != NULL) {
+                                       o_addqchr(dest, i_getch(input), 1);
+                               } else {
+                                       o_addqchr(dest, '\\', 1);
+                               }
+                       } else {
+                               o_addchr(dest, '\\');
+                               o_addchr(dest, i_getch(input));
+                       }
                        break;
                case '$':
-                       if (handle_dollar(dest, /*ctx,*/ input) != 0) {
+                       if (handle_dollar(dest, input) != 0) {
                                debug_printf_parse("parse_stream return 1: handle_dollar returned non-0\n");
                                return 1;
                        }
@@ -3519,10 +3694,10 @@ static int parse_stream(o_string *dest, struct p_context *ctx,
                case '\'':
                        dest->nonnull = 1;
                        while (1) {
-                               ch = b_getch(input);
+                               ch = i_getch(input);
                                if (ch == EOF || ch == '\'')
                                        break;
-                               b_addchr(dest, ch);
+                               o_addqchr(dest, ch, 1);
                        }
                        if (ch == EOF) {
                                syntax("unterminated '");
@@ -3535,9 +3710,15 @@ static int parse_stream(o_string *dest, struct p_context *ctx,
                        dest->o_quote ^= 1; /* invert */
                        break;
 #if ENABLE_HUSH_TICK
-               case '`':
-                       process_command_subs(dest, /*ctx,*/ input, "`");
+               case '`': {
+                       //int pos = dest->length;
+                       o_addchr(dest, SPECIAL_VAR_SYMBOL);
+                       o_addchr(dest, dest->o_quote ? 0x80 | '`' : '`');
+                       add_till_backquote(dest, input);
+                       o_addchr(dest, SPECIAL_VAR_SYMBOL);
+                       //bb_error_msg("RES '%s'", dest->data + pos);
                        break;
+               }
 #endif
                case '>':
                        redir_fd = redirect_opt_num(dest);
@@ -3545,7 +3726,7 @@ static int parse_stream(o_string *dest, struct p_context *ctx,
                        redir_style = REDIRECT_OVERWRITE;
                        if (next == '>') {
                                redir_style = REDIRECT_APPEND;
-                               b_getch(input);
+                               i_getch(input);
                        }
 #if 0
                        else if (next == '(') {
@@ -3562,10 +3743,10 @@ static int parse_stream(o_string *dest, struct p_context *ctx,
                        redir_style = REDIRECT_INPUT;
                        if (next == '<') {
                                redir_style = REDIRECT_HEREIS;
-                               b_getch(input);
+                               i_getch(input);
                        } else if (next == '>') {
                                redir_style = REDIRECT_IO;
-                               b_getch(input);
+                               i_getch(input);
                        }
 #if 0
                        else if (next == '(') {
@@ -3583,7 +3764,7 @@ static int parse_stream(o_string *dest, struct p_context *ctx,
                case '&':
                        done_word(dest, ctx);
                        if (next == '&') {
-                               b_getch(input);
+                               i_getch(input);
                                done_pipe(ctx, PIPE_AND);
                        } else {
                                done_pipe(ctx, PIPE_BG);
@@ -3592,7 +3773,7 @@ static int parse_stream(o_string *dest, struct p_context *ctx,
                case '|':
                        done_word(dest, ctx);
                        if (next == '|') {
-                               b_getch(input);
+                               i_getch(input);
                                done_pipe(ctx, PIPE_OR);
                        } else {
                                /* we could pick up a file descriptor choice here
@@ -3692,14 +3873,14 @@ static int parse_and_run_stream(struct in_str *inp, int parse_flag)
                } else {
                        if (ctx.old_flag != 0) {
                                free(ctx.stack);
-                               b_reset(&temp);
+                               o_reset(&temp);
                        }
                        temp.nonnull = 0;
                        temp.o_quote = 0;
                        inp->p = NULL;
                        free_pipe_list(ctx.list_head, /* indent: */ 0);
                }
-               b_free(&temp);
+               o_free(&temp);
        } while (rcode != -1 && !(parse_flag & PARSEFLAG_EXIT_FROM_LOOP));   /* loop on syntax errors, return on EOF */
        return 0;
 }
@@ -3906,8 +4087,10 @@ int hush_main(int argc, char **argv)
                                /* give up */
                                interactive_fd = 0;
                }
-               if (interactive_fd)
+               if (interactive_fd) {
                        fcntl(interactive_fd, F_SETFD, FD_CLOEXEC);
+                       set_misc_sighandler(SIG_IGN);
+               }
        }
 #endif