From 385cc59117f2788d24d2cd75fd58f7ff044d501c Mon Sep 17 00:00:00 2001 From: Denys Vlasenko Date: Tue, 12 Jan 2010 06:47:39 +0100 Subject: [PATCH] hush: plug a memory leak Signed-off-by: Denys Vlasenko --- shell/hush.c | 1 + .../hush-z_slow/leak_empty_tick.right | 3 ++ .../hush-z_slow/leak_empty_tick.tests | 28 +++++++++++++++++++ 3 files changed, 32 insertions(+) create mode 100644 shell/hush_test/hush-z_slow/leak_empty_tick.right create mode 100755 shell/hush_test/hush-z_slow/leak_empty_tick.tests diff --git a/shell/hush.c b/shell/hush.c index 25094654d..d02e68d49 100644 --- a/shell/hush.c +++ b/shell/hush.c @@ -4136,6 +4136,7 @@ static NOINLINE int run_pipe(struct pipe *pi) /* if someone gives us an empty string: `cmd with empty output` */ if (!argv_expanded[0]) { + free(argv_expanded); debug_leave(); return G.last_exitcode; } diff --git a/shell/hush_test/hush-z_slow/leak_empty_tick.right b/shell/hush_test/hush-z_slow/leak_empty_tick.right new file mode 100644 index 000000000..c6f0334f3 --- /dev/null +++ b/shell/hush_test/hush-z_slow/leak_empty_tick.right @@ -0,0 +1,3 @@ +Warm up +Measuring memory leak... +Ok diff --git a/shell/hush_test/hush-z_slow/leak_empty_tick.tests b/shell/hush_test/hush-z_slow/leak_empty_tick.tests new file mode 100755 index 000000000..ae3757969 --- /dev/null +++ b/shell/hush_test/hush-z_slow/leak_empty_tick.tests @@ -0,0 +1,28 @@ +echo "Warm up" +i=1 +while test $i != 9; do + `true` + : $((i++)) +done + +memleak + +echo "Measuring memory leak..." +i=1 +while test $i != 199; do + `true` + : $((i++)) +done +i=1 +while test $i != 199; do + `true` + : $((i++)) +done + +memleak +kb=$? +if test $kb -le 4; then + echo Ok #$kb +else + echo "Bad: $kb kb (or more) leaked" +fi