hush: handle backslash-newline in heredoc terminators
authorDenys Vlasenko <vda.linux@googlemail.com>
Tue, 24 Jul 2018 12:03:18 +0000 (14:03 +0200)
committerDenys Vlasenko <vda.linux@googlemail.com>
Tue, 24 Jul 2018 12:03:18 +0000 (14:03 +0200)
function                                             old     new   delta
fetch_heredocs                                       479     527     +48

(ash fails this test)

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
shell/ash_test/ash-heredoc/heredoc_bkslash_newline2.right [new file with mode: 0644]
shell/ash_test/ash-heredoc/heredoc_bkslash_newline2.tests [new file with mode: 0755]
shell/hush.c
shell/hush_test/hush-heredoc/heredoc_bkslash_newline2.right [new file with mode: 0644]
shell/hush_test/hush-heredoc/heredoc_bkslash_newline2.tests [new file with mode: 0755]

diff --git a/shell/ash_test/ash-heredoc/heredoc_bkslash_newline2.right b/shell/ash_test/ash-heredoc/heredoc_bkslash_newline2.right
new file mode 100644 (file)
index 0000000..3d79316
--- /dev/null
@@ -0,0 +1 @@
+Ok1
diff --git a/shell/ash_test/ash-heredoc/heredoc_bkslash_newline2.tests b/shell/ash_test/ash-heredoc/heredoc_bkslash_newline2.tests
new file mode 100755 (executable)
index 0000000..1d2a265
--- /dev/null
@@ -0,0 +1,4 @@
+cat <<EOF
+Ok1
+EO\
+F
index 75bce337a0fe374dc4340c29f58c9afa4ac14e14..c26484b49f2d4ede169c94bb722ea8bd391f14fd 100644 (file)
@@ -4250,6 +4250,7 @@ static char *fetch_till_str(o_string *as_string,
                if (ch == '\n' || ch == EOF) {
  check_heredoc_end:
                        if ((heredoc_flags & HEREDOC_QUOTED) || prev != '\\') {
+                               /* End-of-line, and not a line continuation */
                                if (strcmp(heredoc.data + past_EOL, word) == 0) {
                                        heredoc.data[past_EOL] = '\0';
                                        debug_printf_heredoc("parsed '%s' heredoc '%s'\n", word, heredoc.data);
@@ -4275,17 +4276,32 @@ static char *fetch_till_str(o_string *as_string,
                                        if (ch == '\n')
                                                goto check_heredoc_end;
                                }
+                       } else {
+                               /* Backslash-line continuation in an unquoted
+                                * heredoc. This does not need special handling
+                                * for heredoc body (unquoted heredocs are
+                                * expanded on "execution" and that would take
+                                * care of this case too), but not the case
+                                * of line continuation *in terminator*:
+                                *  cat <<EOF
+                                *  Ok1
+                                *  EO\
+                                *  F
+                                */
+                               heredoc.data[--heredoc.length] = '\0';
+                               prev = 0; /* not '\' */
+                               continue;
                        }
                }
                if (ch == EOF) {
                        o_free(&heredoc);
-                       return NULL;
+                       return NULL; /* error */
                }
                o_addchr(&heredoc, ch);
                nommu_addchr(as_string, ch);
                if (prev == '\\' && ch == '\\')
                        /* Correctly handle foo\\<eol> (not a line cont.) */
-                       prev = 0; /* not \ */
+                       prev = 0; /* not '\' */
                else
                        prev = ch;
        }
diff --git a/shell/hush_test/hush-heredoc/heredoc_bkslash_newline2.right b/shell/hush_test/hush-heredoc/heredoc_bkslash_newline2.right
new file mode 100644 (file)
index 0000000..3d79316
--- /dev/null
@@ -0,0 +1 @@
+Ok1
diff --git a/shell/hush_test/hush-heredoc/heredoc_bkslash_newline2.tests b/shell/hush_test/hush-heredoc/heredoc_bkslash_newline2.tests
new file mode 100755 (executable)
index 0000000..1d2a265
--- /dev/null
@@ -0,0 +1,4 @@
+cat <<EOF
+Ok1
+EO\
+F