From b24e55da84093dd6bf6dff79627e32459c3da071 Mon Sep 17 00:00:00 2001 From: Denys Vlasenko Date: Sun, 16 Jul 2017 20:29:35 +0200 Subject: hush: fix "cmd1 && cmd2 &" handling on NOMMU function old new delta done_pipe 234 238 +4 Signed-off-by: Denys Vlasenko --- shell/hush.c | 22 +++++++++++++--------- 1 file changed, 13 insertions(+), 9 deletions(-) (limited to 'shell/hush.c') diff --git a/shell/hush.c b/shell/hush.c index 2a734f3de..8d9292ed1 100644 --- a/shell/hush.c +++ b/shell/hush.c @@ -3378,18 +3378,15 @@ static void done_pipe(struct parse_context *ctx, pipe_style type) ctx->ctx_inverted = 0; ctx->pipe->res_word = ctx->ctx_res_w; #endif - if (type != PIPE_BG || ctx->list_head == ctx->pipe) { - no_conv: - ctx->pipe->followup = type; - } else { - /* Necessary since && and || have more precedence than &: + if (type == PIPE_BG && ctx->list_head != ctx->pipe) { + /* Necessary since && and || have precedence over &: * "cmd1 && cmd2 &" must spawn both cmds, not only cmd2, * in a backgrounded subshell. */ struct pipe *pi; struct command *command; - /* Is this actually the case? */ + /* Is this actually this construct, all pipes end with && or ||? */ pi = ctx->list_head; while (pi != ctx->pipe) { if (pi->followup != PIPE_AND && pi->followup != PIPE_OR) @@ -3408,11 +3405,16 @@ static void done_pipe(struct parse_context *ctx, pipe_style type) command->cmd_type = CMD_NORMAL; command->group = ctx->list_head; #if !BB_MMU -//TODO: is this correct?! - command->group_as_string = xstrdup(ctx->as_string.data); + command->group_as_string = xstrndup( + ctx->as_string.data, + ctx->as_string.length - 1 /* do not copy last char, "&" */ + ); #endif /* Replace all pipes in ctx with one newly created */ ctx->list_head = ctx->pipe = pi; + } else { + no_conv: + ctx->pipe->followup = type; } /* Without this check, even just on command line generates @@ -4855,7 +4857,9 @@ static struct pipe *parse_stream(char **pstring, * Really, ask yourself, why * "cmd && " doesn't start * cmd but waits for more input? - * No reason...) + * The only reason is that it might be + * a "cmd1 && cmd2 &" construct, + * cmd1 may need to run in BG). */ struct pipe *pi = ctx.list_head; if (pi->num_cmds != 0 /* check #1 */ -- cgit v1.2.3