* [LTP] [PATCH 2/2] close_range02: Add simpler test and Check invalid params
@ 2021-02-15 15:53 Richard Palethorpe
2021-02-16 6:48 ` Petr Vorel
0 siblings, 1 reply; 2+ messages in thread
From: Richard Palethorpe @ 2021-02-15 15:53 UTC (permalink / raw)
To: ltp
This adds some coverage for invalid parameters and a min fd above or
on the limits. There is also some overlap with close_range01, but this
test is simpler and has less requirements.
Signed-off-by: Richard Palethorpe <rpalethorpe@suse.com>
---
This does not include a check for EMFILE during unsharing because it
only happens if the _system's_ maximum file count is reduced after
clone and before unshare. This should probably go into a separate
test. Also it seems like a corner case which is not likely to happen
in practice and can not be done without CAP_SYS_ADMIN AFAICT.
More tests could be added, but I think we can leave it to the fuzzers
to find where coverage is lacking now that we have the basics. Also
Cyrils and I discussed creating generic data copying tests which copy
data between various types of FD (file system files, pipes, sockets,
etc.) using various system calls (e.g (vm)splice, send/recvmsg,
write) and these could also use close_range() in addtion to close(). I
think that would be more productive than adding just to
close_range.
runtest/syscalls | 1 +
.../kernel/syscalls/close_range/.gitignore | 1 +
.../syscalls/close_range/close_range02.c | 110 ++++++++++++++++++
3 files changed, 112 insertions(+)
create mode 100644 testcases/kernel/syscalls/close_range/close_range02.c
diff --git a/runtest/syscalls b/runtest/syscalls
index c71f2b371..ae47a6d5e 100644
--- a/runtest/syscalls
+++ b/runtest/syscalls
@@ -121,6 +121,7 @@ close02 close02
close08 close08
close_range01 close_range01
+close_range02 close_range02
confstr01 confstr01
diff --git a/testcases/kernel/syscalls/close_range/.gitignore b/testcases/kernel/syscalls/close_range/.gitignore
index dddc1b7f8..4b8bab442 100644
--- a/testcases/kernel/syscalls/close_range/.gitignore
+++ b/testcases/kernel/syscalls/close_range/.gitignore
@@ -1 +1,2 @@
close_range01
+close_range02
\ No newline at end of file
diff --git a/testcases/kernel/syscalls/close_range/close_range02.c b/testcases/kernel/syscalls/close_range/close_range02.c
new file mode 100644
index 000000000..9e099cec4
--- /dev/null
+++ b/testcases/kernel/syscalls/close_range/close_range02.c
@@ -0,0 +1,110 @@
+// SPDX-License-Identifier: GPL-2.0
+/*
+ * Copyright (c) 2021 SUSE LLC
+ */
+/*\
+ * [DESCRIPTION]
+ *
+ * - First check close_range works on a valid range.
+ * - Then check close_range does not accept invalid paramters.
+ * - Then check it accepts a large lower fd.
+ * - Finally check CLOEXEC works
+ *
+\*/
+
+#include <stdlib.h>
+
+#include "tst_test.h"
+#include "tst_clone.h"
+#include "lapi/fcntl.h"
+#include "lapi/close_range.h"
+#include "lapi/clone.h"
+
+static int try_close_range(int fd, int flags)
+{
+ int res;
+
+ TEST(close_range(fd, fd, flags));
+
+ if (TST_RET == -1 && TST_ERR == EINVAL)
+ res = TCONF;
+ else if (TST_RET == -1)
+ res = TFAIL;
+ else
+ res = TPASS;
+
+ return res;
+}
+
+static void run(unsigned int n)
+{
+ const struct tst_clone_args args = {
+ .flags = CLONE_FILES,
+ .exit_signal = SIGCHLD,
+ };
+ int fd, res;
+
+ switch (n) {
+ case 0:
+ fd = SAFE_OPEN("/", O_PATH);
+ SAFE_DUP2(fd, 100);
+
+ TST_EXP_PASS(close_range(fd, 100, 0),
+ "close_range(%d, 100, 0)", fd);
+ TST_EXP_FAIL(fcntl(fd, F_GETFD), EBADF,
+ "fcntl(%d, F_GETFD)", fd);
+ TST_EXP_FAIL(fcntl(100, F_GETFD), EBADF);
+ break;
+ case 1:
+ TST_EXP_FAIL(close_range(4, 3, 0), EINVAL);
+ break;
+ case 2:
+ TST_EXP_FAIL(close_range(3, ~0U, ~0U), EINVAL);
+ break;
+ case 3:
+ TST_EXP_PASS(close_range(~0U, ~0U, 0));
+ break;
+ case 4:
+ fd = SAFE_OPEN("/", O_PATH);
+
+ res = try_close_range(fd, CLOSE_RANGE_CLOEXEC);
+ tst_res(res | TTERRNO,
+ "close_range(%d, %d, CLOSE_RANGE_CLOEXEC)", fd, fd);
+
+ if (res != TPASS)
+ break;
+
+ TST_EXP_FD_SILENT(fcntl(fd, F_GETFD), "fcntl(%d, F_GETFD)", fd);
+ if (TST_RET & FD_CLOEXEC)
+ tst_res(TPASS, "FD_CLOEXEC was set on %d", fd);
+ else
+ tst_res(TFAIL, "FD_CLOEXEC not set on %d", fd);
+ break;
+ case 5:
+ fd = SAFE_OPEN("/", O_PATH);
+
+ if (!SAFE_CLONE(&args)) {
+ res = try_close_range(fd, CLOSE_RANGE_UNSHARE);
+ tst_res(res | TTERRNO,
+ "close_range(%d, %d, CLOSE_RANGE_UNSHARE)",
+ fd, fd);
+
+ if (res != TPASS)
+ exit(0);
+
+ TST_EXP_FAIL(fcntl(fd, F_GETFD), EBADF,
+ "fcntl(%d, F_GETFD)", fd);
+ exit(0);
+ }
+
+ tst_reap_children();
+
+ TST_EXP_PASS(fcntl(fd, F_GETFD), "%d is open", fd);
+ }
+}
+
+static struct tst_test test = {
+ .tcnt = 6,
+ .forks_child = 1,
+ .test = run,
+};
--
2.30.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [LTP] [PATCH 2/2] close_range02: Add simpler test and Check invalid params
2021-02-15 15:53 [LTP] [PATCH 2/2] close_range02: Add simpler test and Check invalid params Richard Palethorpe
@ 2021-02-16 6:48 ` Petr Vorel
0 siblings, 0 replies; 2+ messages in thread
From: Petr Vorel @ 2021-02-16 6:48 UTC (permalink / raw)
To: ltp
Hi Richard,
> This adds some coverage for invalid parameters and a min fd above or
> on the limits. There is also some overlap with close_range01, but this
> test is simpler and has less requirements.
Reviewed-by: Petr Vorel <pvorel@suse.cz>
LGTM, just minor nit: could we close file descriptor?
Otherwise it fails on higher -i (> 50). Something like code below?
Kind regards,
Petr
diff --git testcases/kernel/syscalls/close_range/close_range02.c testcases/kernel/syscalls/close_range/close_range02.c
index 9e099cec4..237b3d759 100644
--- testcases/kernel/syscalls/close_range/close_range02.c
+++ testcases/kernel/syscalls/close_range/close_range02.c
@@ -42,7 +42,7 @@ static void run(unsigned int n)
.flags = CLONE_FILES,
.exit_signal = SIGCHLD,
};
- int fd, res;
+ int fd = -1, res;
switch (n) {
case 0:
@@ -101,6 +101,9 @@ static void run(unsigned int n)
TST_EXP_PASS(fcntl(fd, F_GETFD), "%d is open", fd);
}
+
+ if (n != 0 && fd > 0)
+ SAFE_CLOSE(fd);
}
static struct tst_test test = {
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-02-16 6:48 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-15 15:53 [LTP] [PATCH 2/2] close_range02: Add simpler test and Check invalid params Richard Palethorpe
2021-02-16 6:48 ` Petr Vorel
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.