Commit 1d44d0dd authored by Tommi Rantala's avatar Tommi Rantala Committed by Shuah Khan
Browse files

selftests: core: use SKIP instead of XFAIL in close_range_test.c 



XFAIL is gone since commit 9847d24a ("selftests/harness: Refactor XFAIL
into SKIP"), use SKIP instead.

Fixes: 9847d24a ("selftests/harness: Refactor XFAIL into SKIP")
Signed-off-by: default avatarTommi Rantala <tommi.t.rantala@nokia.com>
Reviewed-by: default avatarKees Cook <keescook@chromium.org>
Acked-by: default avatarChristian Brauner <christian.brauner@ubuntu.com>
Signed-off-by: default avatarShuah Khan <skhan@linuxfoundation.org>
parent f3ae6c6e

tools/testing/selftests/core/close_range_test.c

+4 −4
Original line number Diff line number Diff line
@@ -44,7 +44,7 @@ TEST(close_range) 
		fd = open("/dev/null", O_RDONLY | O_CLOEXEC);

		ASSERT_GE(fd, 0) {

			if (errno == ENOENT)

				XFAIL(return, "Skipping test since /dev/null does not exist");

				SKIP(return, "Skipping test since /dev/null does not exist");

		}



		open_fds[i] = fd;
@@ -52,7 +52,7 @@ TEST(close_range) 


	EXPECT_EQ(-1, sys_close_range(open_fds[0], open_fds[100], -1)) {

		if (errno == ENOSYS)

			XFAIL(return, "close_range() syscall not supported");

			SKIP(return, "close_range() syscall not supported");

	}



	EXPECT_EQ(0, sys_close_range(open_fds[0], open_fds[50], 0));
@@ -108,7 +108,7 @@ TEST(close_range_unshare) 
		fd = open("/dev/null", O_RDONLY | O_CLOEXEC);

		ASSERT_GE(fd, 0) {

			if (errno == ENOENT)

				XFAIL(return, "Skipping test since /dev/null does not exist");

				SKIP(return, "Skipping test since /dev/null does not exist");

		}



		open_fds[i] = fd;
@@ -197,7 +197,7 @@ TEST(close_range_unshare_capped) 
		fd = open("/dev/null", O_RDONLY | O_CLOEXEC);

		ASSERT_GE(fd, 0) {

			if (errno == ENOENT)

				XFAIL(return, "Skipping test since /dev/null does not exist");

				SKIP(return, "Skipping test since /dev/null does not exist");

		}



		open_fds[i] = fd;

CRA Git | Maintained and supported by SUSTech CRA and CCSE