svn commit: [26162] trunk/busybox/shell: hush_test/hush-trap hush_test/hush-z_sl etc...

[vda at busybox.net]

Author: vda
Date: 2009-04-19 23:07:51 +0000 (Sun, 19 Apr 2009)
New Revision: 26162

Log:
hush: fix "export not_yet_defined_var", fix parsing of "cmd | }"
 corner case; improve hush_leaktool.sh;
 fix some false positives in testsuite

function                                             old     new   delta
builtin_export                                       191     206     +15
parse_stream                                        2196    2200      +4



Modified:
   trunk/busybox/shell/hush.c
   trunk/busybox/shell/hush_leaktool.sh
   trunk/busybox/shell/hush_test/hush-trap/catch.right
   trunk/busybox/shell/hush_test/hush-z_slow/leak_all1.tests


Changeset:
Modified: trunk/busybox/shell/hush.c
===================================================================
--- trunk/busybox/shell/hush.c	2009-04-19 21:37:07 UTC (rev 26161)
+++ trunk/busybox/shell/hush.c	2009-04-19 23:07:51 UTC (rev 26162)
@@ -4459,10 +4459,6 @@
 			}
 		}
 		command->argv = add_string_to_strings(command->argv, xstrdup(word->data));
-//SEGV, but good idea.
-//		command->argv = add_string_to_strings(command->argv, word->data);
-//		word->data = NULL;
-//		word->length = 0;
 		debug_print_strings("word appended to argv", command->argv);
 	}
 
@@ -5481,14 +5477,16 @@
 		 * } is an ordinary char in this case, even inside { cmd; }
 		 * Pathological example: { ""}; } should exec "}" cmd
 		 */
-		if (ch == '}'
-		 && !(IS_NULL_PIPE(ctx.pipe)
-		     && IS_NULL_CMD(ctx.command)
-		     && dest.length == 0
-		     && !dest.o_quoted
-		    )
-		) {
-			goto ordinary_char;
+		if (ch == '}') {
+			if (!IS_NULL_CMD(ctx.command) /* cmd } */
+			 || dest.length != 0 /* word} */
+			 || dest.o_quoted    /* ""} */
+			) {
+				goto ordinary_char;
+			}
+			if (!IS_NULL_PIPE(ctx.pipe)) /* cmd | } */
+				goto skip_end_trigger;
+			/* else: } does terminate a group */
 		}
 
 		if (end_trigger && end_trigger == ch
@@ -5531,6 +5529,7 @@
 				return ctx.list_head;
 			}
 		}
+ skip_end_trigger:
 		if (is_ifs)
 			continue;
 
@@ -6420,12 +6419,11 @@
 	}
 
 	do {
-		const char *value;
 		char *name = *argv;
 
-		value = strchr(name, '=');
-		if (!value) {
-			/* They are exporting something without a =VALUE */
+		/* So far we do not check that name is valid */
+		if (strchr(name, '=') == NULL) {
+			/* Exporting a name without a =VALUE */
 			struct variable *var;
 
 			var = get_local_var(name);
@@ -6433,12 +6431,19 @@
 				var->flg_export = 1;
 				debug_printf_env("%s: putenv '%s'\n", __func__, var->varstr);
 				putenv(var->varstr);
+				continue;
 			}
-			/* bash does not return an error when trying to export
-			 * an undefined variable.  Do likewise. */
-			continue;
+			/* Exporting non-existing variable.
+			 * bash does not put it in environment,
+			 * but remembers that it is exported,
+			 * and does put it in env when it is set later.
+			 * We just set it to "" and export. */
+			name = xasprintf("%s=", name);
+		} else {
+			/* Exporting VAR=VALUE */
+			name = xstrdup(name);
 		}
-		set_local_var(xstrdup(name), 1, 0);
+		set_local_var(name, 1, 0);
 	} while (*++argv);
 
 	return EXIT_SUCCESS;

Modified: trunk/busybox/shell/hush_leaktool.sh
===================================================================
--- trunk/busybox/shell/hush_leaktool.sh	2009-04-19 21:37:07 UTC (rev 26161)
+++ trunk/busybox/shell/hush_leaktool.sh	2009-04-19 23:07:51 UTC (rev 26162)
@@ -6,8 +6,20 @@
 freelist=`grep 'free 0x' "$output" | cut -d' ' -f2 | sort | uniq | xargs`
 
 grep -v free "$output" >"$output.leaked"
+
+i=8
+list=
 for freed in $freelist; do
-    echo Dropping $freed
-    grep -v $freed <"$output.leaked" >"$output.temp"
+    list="$list -e $freed"
+    test $((--i)) != 0 && continue
+    echo Dropping $list
+    grep -F -v $list <"$output.leaked" >"$output.temp"
     mv "$output.temp" "$output.leaked"
+    i=8
+    list=
 done
+if test "$list"; then
+    echo Dropping $list
+    grep -F -v $list <"$output.leaked" >"$output.temp"
+    mv "$output.temp" "$output.leaked"
+fi

Modified: trunk/busybox/shell/hush_test/hush-trap/catch.right
===================================================================
--- trunk/busybox/shell/hush_test/hush-trap/catch.right	2009-04-19 21:37:07 UTC (rev 26161)
+++ trunk/busybox/shell/hush_test/hush-trap/catch.right	2009-04-19 23:07:51 UTC (rev 26162)
@@ -2,3 +2,4 @@
 caught
 sending USR2
 sending USR2
+USR2

Modified: trunk/busybox/shell/hush_test/hush-z_slow/leak_all1.tests
===================================================================
--- trunk/busybox/shell/hush_test/hush-z_slow/leak_all1.tests	2009-04-19 21:37:07 UTC (rev 26161)
+++ trunk/busybox/shell/hush_test/hush-z_slow/leak_all1.tests	2009-04-19 23:07:51 UTC (rev 26162)
@@ -67,6 +67,7 @@
     f >/dev/null
     : $((i++))
 done
+unset -f f
 
 memleak
 
@@ -133,6 +134,7 @@
     f >/dev/null
     : $((i++))
 done
+unset -f f
 
 
 memleak