svn commit: [26021] trunk/busybox/shell: hush_test/hush-misc
vda at busybox.net
vda at busybox.net
Wed Apr 8 09:29:15 UTC 2009
Author: vda
Date: 2009-04-08 09:29:14 +0000 (Wed, 08 Apr 2009)
New Revision: 26021
Log:
hush: echo \2>file fix
Added:
trunk/busybox/shell/hush_test/hush-misc/redir2.right
trunk/busybox/shell/hush_test/hush-misc/redir2.tests
Modified:
trunk/busybox/shell/hush.c
trunk/busybox/shell/hush_test/hush-misc/heredoc2.right
trunk/busybox/shell/hush_test/hush-misc/heredoc2.tests
trunk/busybox/shell/hush_test/hush-misc/heredoc3.right
trunk/busybox/shell/hush_test/hush-misc/heredoc3.tests
Changeset:
Modified: trunk/busybox/shell/hush.c
===================================================================
--- trunk/busybox/shell/hush.c 2009-04-07 21:04:31 UTC (rev 26020)
+++ trunk/busybox/shell/hush.c 2009-04-08 09:29:14 UTC (rev 26021)
@@ -4081,7 +4081,7 @@
* the redirection expression. For example:
* echo \2>a
* writes the character 2 into file a"
- * I am not sure we do it right (and not sure we care)
+ * We are getting it right by setting ->o_quoted on any \<char>
*
* A -1 return means no valid number was found,
* the caller should use the appropriate default for this redirection.
@@ -4141,6 +4141,9 @@
}
}
+/* Look at entire parse tree for not-yet-loaded REDIRECT_HEREDOCs
+ * and load them all. There should be exactly heredoc_cnt of them.
+ */
static int fetch_heredocs(int heredoc_cnt, struct parse_context *ctx, struct in_str *input)
{
struct pipe *pi = ctx->list_head;
@@ -4161,14 +4164,18 @@
if (redir->rd_type == REDIRECT_HEREDOC) {
char *p;
- if (heredoc_cnt <= 0)
+ if (heredoc_cnt <= 0) {
+ syntax("heredoc BUG 1");
return 1; /* error */
+ }
redir->rd_type = REDIRECT_HEREDOC2;
/* redir->dup is (ab)used to indicate <<- */
p = fetch_till_str(&ctx->as_string, input,
redir->rd_filename, redir->rd_dup & HEREDOC_SKIPTABS);
- if (!p)
- return 1; /* unexpected EOF */
+ if (!p) {
+ syntax("unexpected EOF in here document");
+ return 1;
+ }
free(redir->rd_filename);
redir->rd_filename = p;
heredoc_cnt--;
@@ -4180,6 +4187,8 @@
pi = pi->next;
}
/* Should be 0. If it isn't, it's a parse error */
+ if (heredoc_cnt)
+ syntax("heredoc BUG 2");
return heredoc_cnt;
}
@@ -4673,12 +4682,12 @@
if (ch != EOF)
nommu_addchr(as_string, ch);
if (ch == dquote_end) { /* may be only '"' or EOF */
- dest->o_quoted = 1;
if (dest->o_assignment == NOT_ASSIGNMENT)
dest->o_escape ^= 1;
debug_printf_parse("parse_stream_dquoted return 0\n");
return 0;
}
+ /* note: can't move it above ch == dquote_end check! */
if (ch == EOF) {
syntax("unterminated \"");
debug_printf_parse("parse_stream_dquoted return 1: unterminated \"\n");
@@ -4787,6 +4796,7 @@
redir_type redir_style;
if (is_in_dquote) {
+ /* dest.o_quoted = 1; - already is (see below) */
if (parse_stream_dquoted(&ctx.as_string, &dest, input, '"')) {
goto parse_error;
}
@@ -4863,8 +4873,9 @@
done_pipe(&ctx, PIPE_SEQ);
debug_printf_parse("heredoc_cnt:%d\n", heredoc_cnt);
if (heredoc_cnt) {
- if (fetch_heredocs(heredoc_cnt, &ctx, input))
+ if (fetch_heredocs(heredoc_cnt, &ctx, input)) {
goto parse_error;
+ }
heredoc_cnt = 0;
}
dest.o_assignment = MAYBE_ASSIGNMENT;
@@ -4938,7 +4949,6 @@
i_getch(input);
/* note: we do not add it to &ctx.as_string */
}
-//TODO: go back one char?
nommu_addchr(&ctx.as_string, '\n');
} else {
o_addQchr(&dest, ch);
@@ -4951,8 +4961,11 @@
}
o_addchr(&dest, '\\');
ch = i_getch(input);
+ nommu_addchr(&ctx.as_string, ch);
o_addchr(&dest, ch);
- nommu_addchr(&ctx.as_string, ch);
+ /* Example: echo Hello \2>file
+ * we need to know that word 2 is quoted */
+ dest.o_quoted = 1;
break;
case '$':
if (handle_dollar(&ctx.as_string, &dest, input) != 0) {
Modified: trunk/busybox/shell/hush_test/hush-misc/heredoc2.right
===================================================================
--- trunk/busybox/shell/hush_test/hush-misc/heredoc2.right 2009-04-07 21:04:31 UTC (rev 26020)
+++ trunk/busybox/shell/hush_test/hush-misc/heredoc2.right 2009-04-08 09:29:14 UTC (rev 26021)
@@ -6,3 +6,4 @@
EOF-f
EOF-f f
EOF-f
+Ok
Modified: trunk/busybox/shell/hush_test/hush-misc/heredoc2.tests
===================================================================
--- trunk/busybox/shell/hush_test/hush-misc/heredoc2.tests 2009-04-07 21:04:31 UTC (rev 26020)
+++ trunk/busybox/shell/hush_test/hush-misc/heredoc2.tests 2009-04-08 09:29:14 UTC (rev 26021)
@@ -9,3 +9,4 @@
EOF-f f
EOF-f
EOF-f
+echo Ok
Modified: trunk/busybox/shell/hush_test/hush-misc/heredoc3.right
===================================================================
--- trunk/busybox/shell/hush_test/hush-misc/heredoc3.right 2009-04-07 21:04:31 UTC (rev 26020)
+++ trunk/busybox/shell/hush_test/hush-misc/heredoc3.right 2009-04-08 09:29:14 UTC (rev 26021)
@@ -6,3 +6,4 @@
EOF-f
EOF-f f
EOF-f
+Ok
Modified: trunk/busybox/shell/hush_test/hush-misc/heredoc3.tests
===================================================================
--- trunk/busybox/shell/hush_test/hush-misc/heredoc3.tests 2009-04-07 21:04:31 UTC (rev 26020)
+++ trunk/busybox/shell/hush_test/hush-misc/heredoc3.tests 2009-04-08 09:29:14 UTC (rev 26021)
@@ -9,3 +9,4 @@
EOF-f f
EOF-f
EOF-f
+echo Ok
Added: trunk/busybox/shell/hush_test/hush-misc/redir2.right
===================================================================
--- trunk/busybox/shell/hush_test/hush-misc/redir2.right (rev 0)
+++ trunk/busybox/shell/hush_test/hush-misc/redir2.right 2009-04-08 09:29:14 UTC (rev 26021)
@@ -0,0 +1 @@
+Ok
Added: trunk/busybox/shell/hush_test/hush-misc/redir2.tests
===================================================================
--- trunk/busybox/shell/hush_test/hush-misc/redir2.tests (rev 0)
+++ trunk/busybox/shell/hush_test/hush-misc/redir2.tests 2009-04-08 09:29:14 UTC (rev 26021)
@@ -0,0 +1,2 @@
+echo NOT SHOWN \2>/dev/null
+echo Ok
Property changes on: trunk/busybox/shell/hush_test/hush-misc/redir2.tests
___________________________________________________________________
Added: svn:executable
+ *
More information about the busybox-cvs
mailing list