shells: update README and testsuite. environment memory leak
authorDenis Vlasenko <vda.linux@googlemail.com>
Mon, 21 May 2007 10:19:48 +0000 (10:19 -0000)
committerDenis Vlasenko <vda.linux@googlemail.com>
Mon, 21 May 2007 10:19:48 +0000 (10:19 -0000)
needs to be fixed - and documented!

shell/README
shell/hush_test/hush-parsing/process_subst.right
shell/hush_test/hush-parsing/process_subst.tests

index d492671fb286c10c224d4196ed4b4cdf214f7b4f..919cb9feffa69dbcd27c3067835c6c98f2f770f1 100644 (file)
@@ -1,6 +1,20 @@
 Various bits of what is known about busybox shells, in no particular order.
 
-2006-05-06
+2007-05-21
+hush: environment cannot be handled by libc routines as they are leaky
+(by API design and thus unfixable): hush will leak memory in this script,
+bash does not:
+pid=$$
+while true; do
+    unset t;
+    t=111111111111111111111111111111111111111111111111111111111111111111111111
+    export t
+    ps -o vsz,pid,comm | grep " $pid "
+done
+The fix is to not use setenv/putenv/unsetenv but manipulate env ourself. TODO.
+hush: meanwhile, first three command subst bugs mentioned below are fixed. :)
+
+2007-05-06
 hush: more bugs spotted. Comparison with bash:
 bash-3.2# echo "TEST`date;echo;echo`BEST"
 TESTSun May  6 09:21:05 CEST 2007BEST         [we dont strip eols]
index 8f9ab9d4088237381d7213f02b72258134c24553..397bc8067bc7eb803e866a3bc7796e1823e54158 100644 (file)
@@ -1,2 +1,3 @@
 TESTzzBEST
 TEST$(echo zz)BEST
+TEST'BEST
index f8299a514586da5396f9accbf3869d6689436de9..21996bc0e6f5be119b8d0ab20bfae36d62ad7156 100755 (executable)
@@ -1,2 +1,3 @@
 echo "TEST`echo zz;echo;echo`BEST"
 echo "TEST`echo '$(echo zz)'`BEST"
+echo "TEST`echo "'"`BEST"