[PATCH] selinux-testsuite: fix test/file to avoid noise in test output

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



commit ec3ea0fcf3742cba319 ("tests/file: use getfattr instead of ls -Z")
introduced noise in the test output for the file tests because the
context value returned by getfattr includes the terminating NUL and
this caused the command lines to be truncated, omitting the stderr
redirection.  Use the same technique we already use in overlay/test
by chop()'ing off the terminating NUL.

Signed-off-by: Stephen Smalley <sds@xxxxxxxxxxxxx>
---
 tests/file/test | 1 +
 1 file changed, 1 insertion(+)

diff --git a/tests/file/test b/tests/file/test
index 8fe98f6a5b4a..5e080fc6d4e9 100755
--- a/tests/file/test
+++ b/tests/file/test
@@ -51,6 +51,7 @@ system "chcon -t fileop_exec_t $basedir/wait_io 2>&1 > /dev/null";
 #
 $good_file_sid =
   `getfattr --only-values -n security.selinux $basedir/temp_file`;
+chop($good_file_sid);
 
 #
 # Attempt to access a restricted file as the 'good' domain.  The first test
-- 
2.21.0




[Index of Archives]     [Selinux Refpolicy]     [Linux SGX]     [Fedora Users]     [Fedora Desktop]     [Yosemite Photos]     [Yosemite Camping]     [Yosemite Campsites]     [KDE Users]     [Gnome Users]

  Powered by Linux