diff options
author | Akinobu Mita <akinobu.mita@gmail.com> | 2012-07-31 19:41:31 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-07-31 21:42:38 -0400 |
commit | d6b09e754c23b657544f6e7f39fbf7de24c58aa2 (patch) | |
tree | 7f1128c531b80c968e503327f0d65b27cd3c3764 /tools | |
parent | 2e3ee613480563a6d5c01b57d342e65cc58c06df (diff) |
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')
-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 1776e924b202..78a9ed7fecdb 100644 --- a/tools/testing/fault-injection/failcmd.sh +++ b/tools/testing/fault-injection/failcmd.sh | |||
@@ -206,7 +206,7 @@ while true; do | |||
206 | esac | 206 | esac |
207 | done | 207 | done |
208 | 208 | ||
209 | [ -z "$@" ] && exit 0 | 209 | [ -z "$1" ] && exit 0 |
210 | 210 | ||
211 | echo $oom_kill_allocating_task > /proc/sys/vm/oom_kill_allocating_task | 211 | echo $oom_kill_allocating_task > /proc/sys/vm/oom_kill_allocating_task |
212 | echo $task_filter > $FAULTATTR/task-filter | 212 | echo $task_filter > $FAULTATTR/task-filter |