svn commit: trunk/busybox/shell: hush_test/hush-bugs

[vda at busybox.net]

Author: vda
Date: 2007-05-20 14:51:38 -0700 (Sun, 20 May 2007)
New Revision: 18658

Log:
hush: make process substitution configurable; add a testcase


Added:
   trunk/busybox/shell/hush_test/hush-bugs/tick.right
   trunk/busybox/shell/hush_test/hush-bugs/tick.tests

Modified:
   trunk/busybox/shell/Config.in
   trunk/busybox/shell/hush.c


Changeset:
Modified: trunk/busybox/shell/Config.in
===================================================================
--- trunk/busybox/shell/Config.in	2007-05-20 17:12:43 UTC (rev 18657)
+++ trunk/busybox/shell/Config.in	2007-05-20 21:51:38 UTC (rev 18658)
@@ -197,6 +197,11 @@
 	  prompting for next command (or executing next command in a script),
 	  but no separate process group is formed.
 
+config HUSH_TICK
+	bool "Process substitution"
+	default n
+	help
+	  Enable process substitution `command` and $(command) in hush.
 
 config LASH
 	bool "lash"

Modified: trunk/busybox/shell/hush.c
===================================================================
--- trunk/busybox/shell/hush.c	2007-05-20 17:12:43 UTC (rev 18657)
+++ trunk/busybox/shell/hush.c	2007-05-20 21:51:38 UTC (rev 18658)
@@ -480,7 +480,9 @@
 /*   primary string parsing: */
 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, 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 char *make_string(char **inp);
@@ -3054,6 +3056,7 @@
 	return num;
 }
 
+#if ENABLE_HUSH_TICK
 static FILE *generate_stream_from_list(struct pipe *head)
 {
 	FILE *pf;
@@ -3131,6 +3134,7 @@
 	debug_printf("pclosed, retcode=%d\n", retcode);
 	return retcode;
 }
+#endif
 
 static int parse_group(o_string *dest, struct p_context *ctx,
 	struct in_str *input, int ch)
@@ -3262,10 +3266,12 @@
 			}
 			b_addchr(dest, SPECIAL_VAR_SYMBOL);
 			break;
+#if ENABLE_HUSH_TICK
 		case '(':
 			b_getch(input);
 			process_command_subs(dest, ctx, input, ")");
 			break;
+#endif
 		case '-':
 		case '_':
 			/* still unhandled, but should be eventually */
@@ -3371,9 +3377,11 @@
 			dest->nonnull = 1;
 			dest->quote = !dest->quote;
 			break;
+#if ENABLE_HUSH_TICK
 		case '`':
 			process_command_subs(dest, ctx, input, "`");
 			break;
+#endif
 		case '>':
 			redir_fd = redirect_opt_num(dest);
 			done_word(dest, ctx);
@@ -3481,9 +3489,13 @@
 	 * and on most machines that would be faster (reduced L1 cache use).
 	 */
 	memset(charmap, CHAR_ORDINARY, sizeof(charmap));
+#if ENABLE_HUSH_TICK
 	set_in_charmap("\\$\"`", CHAR_SPECIAL);
+#else
+	set_in_charmap("\\$\"", CHAR_SPECIAL);
+#endif
 	set_in_charmap("<>;&|(){}#'", CHAR_ORDINARY_IF_QUOTED);
-	set_in_charmap(ifs, CHAR_IFS);  /* also flow through if quoted */
+	set_in_charmap(ifs, CHAR_IFS);  /* are ordinary if quoted */
 }
 
 /* most recursion does not come through here, the exception is

Added: trunk/busybox/shell/hush_test/hush-bugs/tick.right
===================================================================

Added: trunk/busybox/shell/hush_test/hush-bugs/tick.tests
===================================================================
--- trunk/busybox/shell/hush_test/hush-bugs/tick.tests	                        (rev 0)
+++ trunk/busybox/shell/hush_test/hush-bugs/tick.tests	2007-05-20 21:51:38 UTC (rev 18658)
@@ -0,0 +1,4 @@
+true
+false; echo `echo $?`
+true
+{ false; echo `echo $?`; }


Property changes on: trunk/busybox/shell/hush_test/hush-bugs/tick.tests
___________________________________________________________________
Name: svn:executable
   + *