8805eeb44
Denis Vlasenko
2007-05-21
Subject
shells: update README and testsuite. environment memory leak needs to be fixed - and documented!
Binary
No binary (build failed)
Architecture & Compiler Flags
—
Build Log
(log yok)
Commit Diff GitHub →
shell/README +15 -1shell/hush_test/hush-parsing/process_subst.right +1 -0shell/hush_test/hush-parsing/process_subst.tests +1 -0
shell/README+15 -1
| @@ -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] | |
shell/hush_test/hush-parsing/process_subst.right+1 -0
| @@ -1,2 +1,3 @@ | |
| TESTzzBEST | |
| TEST$(echo zz)BEST | |
| + | TEST'BEST |
shell/hush_test/hush-parsing/process_subst.tests+1 -0
| @@ -1,2 +1,3 @@ | |
| echo "TEST`echo zz;echo;echo`BEST" | |
| echo "TEST`echo '$(echo zz)'`BEST" | |
| + | echo "TEST`echo "'"`BEST" |