diff options
| author |   Luis Chamberlain <mcgrof@kernel.org> | 2019-05-14 15:45:04 -0700 |
|---|---|---|
| committer |   Linus Torvalds <torvalds@linux-foundation.org> | 2019-05-14 19:52:51 -0700 |
| commit | 8ded3d1026b26240eaa285b98b7942d27f657355 (patch) | |
| tree | d8adf3c16b9b500e2e76a69a0d9422d12a7510a6 /tools/testing/selftests/sysctl/sysctl.sh | |
| parent | tools/testing/selftests/sysctl/sysctl.sh: load module before testing for it (diff) | |
| download | linux-dev-8ded3d1026b26240eaa285b98b7942d27f657355.tar.xz linux-dev-8ded3d1026b26240eaa285b98b7942d27f657355.zip | |
tools/testing/selftests/sysctl/sysctl.sh: ignore diff output on verify_diff_w()
When verify_diff_w() is used we care about the result, not the verbose
output, and although we use -q, that still gives us a chatty message
about if the files differ or not. Since verify_diff_w() uses stdinput
the chatty message says whether or not "-" matches the target file, and
this just seems rather odd. Better to just ignore that messsage all
together, what we really care about i sthe results, the return value and
we check for that.
Link: http://lkml.kernel.org/r/20190320222831.8243-4-mcgrof@kernel.org
Signed-off-by: Luis Chamberlain <mcgrof@kernel.org>
Acked-by: Kees Cook <keescook@chromium.org>
Cc: Eric Sandeen <sandeen@redhat.com>
Cc: Eric Sandeen <sandeen@sandeen.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'tools/testing/selftests/sysctl/sysctl.sh')
| -rwxr-xr-x | tools/testing/selftests/sysctl/sysctl.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/testing/selftests/sysctl/sysctl.sh b/tools/testing/selftests/sysctl/sysctl.sh index e0c8404da6b0..f51987d0d32d 100755 --- a/tools/testing/selftests/sysctl/sysctl.sh +++ b/tools/testing/selftests/sysctl/sysctl.sh @@ -179,7 +179,7 @@ verify() verify_diff_w() { - echo "$TEST_STR" | diff -q -w -u - $1 + echo "$TEST_STR" | diff -q -w -u - $1 > /dev/null return $? } |