diff options
| author | Akinobu Mita <akinobu.mita@gmail.com> | 2012-07-31 16:41:31 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-07-31 18:42:38 -0700 |
| commit | d6b09e754c23b657544f6e7f39fbf7de24c58aa2 (patch) | |
| tree | 7f1128c531b80c968e503327f0d65b27cd3c3764 /tools/testing | |
| parent | 2e3ee613480563a6d5c01b57d342e65cc58c06df (diff) | |
| download | op-kernel-dev-d6b09e754c23b657544f6e7f39fbf7de24c58aa2.zip op-kernel-dev-d6b09e754c23b657544f6e7f39fbf7de24c58aa2.tar.gz | |
fault-injection: fix failcmd.sh warning
"fault-injection: add tool to run command with failslab or
fail_page_alloc" added tools/testing/fault-injection/failcmd.sh to make it
easier to inject slab/page allocation failures by fault injection.
failcmd.sh prints the following warning when running with arguments
for command.
# ./failcmd.sh echo aaa
failcmd.sh: line 209: [: echo: binary operator expected
aaa
This warning is caused by an improper check whether at least one
parameter is left after parsing command options.
Fix it by testing the length of $1 instead of $@
Signed-off-by: Akinobu Mita <akinobu.mita@gmail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'tools/testing')
| -rw-r--r-- | tools/testing/fault-injection/failcmd.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/testing/fault-injection/failcmd.sh b/tools/testing/fault-injection/failcmd.sh index 1776e92..78a9ed7 100644 --- a/tools/testing/fault-injection/failcmd.sh +++ b/tools/testing/fault-injection/failcmd.sh @@ -206,7 +206,7 @@ while true; do esac done -[ -z "$@" ] && exit 0 +[ -z "$1" ] && exit 0 echo $oom_kill_allocating_task > /proc/sys/vm/oom_kill_allocating_task echo $task_filter > $FAULTATTR/task-filter |
