[LTP] [PATCH v3 3/3] fchownat03: Add error tests for fchownat()
Shiyang Ruan
ruansy.fnst@fujitsu.com
Thu Jan 16 09:40:13 CET 2025
Move the 3 error tests form fchownat01 to here, and add tests for rest
possible errnos per the man page.
Signed-off-by: Shiyang Ruan <ruansy.fnst@fujitsu.com>
---
testcases/kernel/syscalls/fchownat/.gitignore | 1 +
.../kernel/syscalls/fchownat/fchownat03.c | 148 ++++++++++++++++++
2 files changed, 149 insertions(+)
create mode 100644 testcases/kernel/syscalls/fchownat/fchownat03.c
diff --git a/testcases/kernel/syscalls/fchownat/.gitignore b/testcases/kernel/syscalls/fchownat/.gitignore
index 35c00345b..60fac7e69 100644
--- a/testcases/kernel/syscalls/fchownat/.gitignore
+++ b/testcases/kernel/syscalls/fchownat/.gitignore
@@ -1,2 +1,3 @@
/fchownat01
/fchownat02
+/fchownat03
diff --git a/testcases/kernel/syscalls/fchownat/fchownat03.c b/testcases/kernel/syscalls/fchownat/fchownat03.c
new file mode 100644
index 000000000..06928490b
--- /dev/null
+++ b/testcases/kernel/syscalls/fchownat/fchownat03.c
@@ -0,0 +1,148 @@
+// SPDX-License-Identifier: GPL-2.0-or-later
+/*
+ * Copyright (c) International Business Machines Corp., 2006
+ * Copyright (c) Linux Test Project, 2007-2024
+ */
+
+/*\
+ * [Description]
+ *
+ * Verify that,
+ * - fchownat() returns -1 and sets errno to EACCES if there is no permission
+ to access to the file.
+ * - fchownat() returns -1 and sets errno to EBADF if the file descriptor
+ * of the specified file is not valid.
+ * - fchownat() returns -1 and sets errno to EFAULT if the filename points
+ outside of accessable address space.
+ * - fchownat() returns -1 and sets errno to EINVAL if the flag is invalid.
+ * - fchownat() returns -1 and sets errno to ELOOP if too many symbolic links
+ * were encountered in resolving filename.
+ * - fchownat() returns -1 and sets errno to ENAMETOOLONG if the filename is
+ too long.
+ * - fchownat() returns -1 and sets errno to ENOENT if the specified file does
+ * not exist.
+ * - fchownat() returns -1 and sets errno to ENOTDIR if the file descriptor is
+ * a file.
+ * - fchownat() returns -1 and sets errno to EPERM if the effective user id
+ * of process does not match the owner of the file and the process is not
+ * super user.
+ * - fchownat() returns -1 and sets errno to EROFS if the file is readonly.
+ */
+
+#define _GNU_SOURCE
+#include <pwd.h>
+#include "tst_test.h"
+
+#define TESTFILE "testfile"
+#define TESTFILE_EACCESS_DIR "eaccess"
+#define TESTFILE_EACCESS TESTFILE_EACCESS_DIR"/eaccess"
+#define TESTFILE_ELOOP "testfile_eloop"
+#define TESTFILE_ENOENT "/tmp/does/not/exist"
+#define TESTFILE_EPERM "/dev/null"
+#define TESTFILE_EROFS_MNT "ro_mntpoint"
+#define TESTFILE_EROFS TESTFILE_EROFS_MNT"/file"
+
+static int file_fd = -1;
+static int no_fd = -1;
+static int dir_fd = -1;
+
+static char *file_eaccess;
+static char *file_ebadf;
+static char *file_efault;
+static char *file_einval;
+static char *file_eloop;
+static char *file_enametoolong;
+static char *file_enoent;
+static char *file_enotdir;
+static char *file_eperm;
+static char *file_erofs;
+
+static struct tcase {
+ char **filename;
+ char *desc;
+ int *fd;
+ int flag;
+ int exp_errno;
+} tcases[] = {
+ {&file_eaccess, TESTFILE_EACCESS, &dir_fd, 0, EACCES},
+ {&file_ebadf, TESTFILE, &no_fd, 0, EBADF},
+ {&file_efault, "Invalid address", &dir_fd, 0, EFAULT},
+ {&file_einval, TESTFILE, &dir_fd, 9999, EINVAL},
+ {&file_eloop, TESTFILE_ELOOP, &dir_fd, 0, ELOOP},
+ {&file_enametoolong, "aaaa...", &dir_fd, 0, ENAMETOOLONG},
+ {&file_enoent, TESTFILE_ENOENT, &dir_fd, 0, ENOENT},
+ {&file_enotdir, TESTFILE, &file_fd, 0, ENOTDIR},
+ {&file_eperm, TESTFILE_EPERM, &dir_fd, 0, EPERM},
+ {&file_erofs, TESTFILE_EROFS, &dir_fd, 0, EROFS},
+};
+
+static void fchownat_verify(unsigned int n)
+{
+ uid_t euid = geteuid();
+ gid_t egid = getegid();
+
+ TST_EXP_FAIL(fchownat(*tcases[n].fd, *tcases[n].filename, euid, egid, tcases[n].flag),
+ tcases[n].exp_errno,
+ "fchownat(%d, %s, %d, %d, %d)",
+ *tcases[n].fd, tcases[n].desc, euid, egid, tcases[n].flag);
+}
+
+static void setup(void)
+{
+ struct passwd *ltpuser;
+
+ SAFE_TOUCH(TESTFILE, 0600, NULL);
+ dir_fd = SAFE_OPEN("./", O_DIRECTORY);
+
+ /* EACCES setting */
+ SAFE_SETEUID(0);
+ SAFE_MKDIR(TESTFILE_EACCESS_DIR, S_IRWXU);
+ SAFE_TOUCH(TESTFILE_EACCESS, 0666, NULL);
+ ltpuser = SAFE_GETPWNAM("nobody");
+ SAFE_SETEUID(ltpuser->pw_uid);
+
+ /* EFAULT setting */
+ file_efault = tst_get_bad_addr(NULL);
+
+ /* ENOTDIR setting */
+ file_fd = SAFE_OPEN("file_fd", O_CREAT | O_RDWR, 0600);
+
+ /* ELOOP setting */
+ SAFE_SYMLINK(TESTFILE_ELOOP, "test_file_eloop2");
+ SAFE_SYMLINK("test_file_eloop2", TESTFILE_ELOOP);
+
+ /* ENAMETOOLONG setting */
+ memset(file_enametoolong, 'a', PATH_MAX+1);
+ file_enametoolong[PATH_MAX+1] = 0;
+}
+
+static void cleanup(void)
+{
+ if (dir_fd != -1)
+ SAFE_CLOSE(dir_fd);
+ if (file_fd != -1)
+ SAFE_CLOSE(file_fd);
+}
+
+static struct tst_test test = {
+ .needs_tmpdir = 1,
+ .needs_root = 1,
+ .needs_rofs = 1,
+ .mntpoint = TESTFILE_EROFS_MNT,
+ .test = fchownat_verify,
+ .tcnt = ARRAY_SIZE(tcases),
+ .setup = setup,
+ .cleanup = cleanup,
+ .bufs = (struct tst_buffers []) {
+ {&file_eaccess, .str = TESTFILE_EACCESS},
+ {&file_ebadf, .str = TESTFILE},
+ {&file_einval, .str = TESTFILE},
+ {&file_eloop, .str = TESTFILE_ELOOP},
+ {&file_enametoolong, .size = PATH_MAX+2},
+ {&file_enoent, .str = TESTFILE_ENOENT},
+ {&file_enotdir, .str = TESTFILE},
+ {&file_eperm, .str = TESTFILE_EPERM},
+ {&file_erofs, .str = TESTFILE_EROFS},
+ {}
+ },
+};
--
2.43.0
More information about the ltp
mailing list