syscalls/{f,l,}chown: Don't pass undocumented flags to open and chmod

testcases/kernel/syscalls/chmod/.gitignore

@@ -5,3 +5,4 @@
 /chmod05
 /chmod06
 /chmod07
+/chmod08

testcases/kernel/syscalls/chmod/chmod08.c

@@ -0,0 +1,64 @@
+/* SPDX-License-Identifier: GPL-2.0-or-later
+ * Copyright (C) 2020 Invisible Things Lab
+ *                    Michał Kowalczyk <[email protected]>
+ */
+
+/*
+ * DESCRIPTION
+ *  Changes file access permissions using `chmod` with bits outside of 07777 in
+ *  `mode` set and verifies if they were ignored.
+ *
+ * WARNING
+ *  The fact that these bits are ignored is not documented (at the time of
+ *  writing). Failure of this test doesn't necessarily mean that a regression
+ *  in Linux was introduced, its intention is to catch accidental interface
+ *  changes and warn kernel developers if that happens.
+ */
+
+#include <errno.h>
+#include <fcntl.h>
+#include <sys/stat.h>
+#include <sys/types.h>
+
+#include "tst_test.h"
+
+#define OPEN_MODE  0644
+#define CHMOD_MODE (0777 | ~07777)
+#define TESTFILE   "testfile"
+
+void test_chmod(void)
+{
+	struct stat stat_buf;
+
+	TEST(chmod(TESTFILE, CHMOD_MODE));
+	if (TST_RET == -1) {
+		tst_res(TFAIL, "chmod(%s, %#o) failed", TESTFILE, CHMOD_MODE);
+	}
+
+	if (stat(TESTFILE, &stat_buf) == -1) {
+		tst_brk(TFAIL | TTERRNO, "stat failed");
+	}
+
+	mode_t expected = S_IFREG | (CHMOD_MODE & 07777);
+	if (stat_buf.st_mode == expected) {
+		tst_res(TPASS, "Unknown mode bits were ignored as expected",
+				TESTFILE, CHMOD_MODE);
+	} else {
+		tst_res(TFAIL, "%s: Incorrect mode 0%04o, expected 0%04o",
+				TESTFILE, stat_buf.st_mode, expected);
+	}
+}
+
+void setup(void)
+{
+	int fd;
+
+	fd = SAFE_OPEN(TESTFILE, O_RDWR | O_CREAT, OPEN_MODE);
+	SAFE_CLOSE(fd);
+}
+
+static struct tst_test test = {
+	.needs_tmpdir = 1,
+	.setup = setup,
+	.test_all = test_chmod,
+};

testcases/kernel/syscalls/open/.gitignore

@@ -13,3 +13,4 @@
 /open12_child
 /open13
 /open14
+/open15

testcases/kernel/syscalls/open/open15.c

@@ -0,0 +1,67 @@
+/* SPDX-License-Identifier: GPL-2.0-or-later
+ * Copyright (C) 2020 Invisible Things Lab
+ *                    Michał Kowalczyk <[email protected]>
+ */
+
+/*
+ * DESCRIPTION
+ *  Creates a file using `open` with bits outside of 07777 in `mode` set and
+ *  verifies if they were ignored.
+ *
+ * WARNING
+ *  The fact that these bits are ignored is not documented (at the time of
+ *  writing). Failure of this test doesn't necessarily mean that a regression
+ *  in Linux was introduced, its intention is to catch accidental interface
+ *  changes and warn kernel developers if that happens.
+ */
+
+#include <errno.h>
+#include <fcntl.h>
+#include <sys/stat.h>
+#include <sys/types.h>
+
+#include "tst_test.h"
+
+#define TEST_FILE "testfile"
+
+static int fd;
+
+static struct tcase {
+	char *filename;
+	int flags;
+	mode_t mode;
+} tcases[] = {
+	{TEST_FILE, O_RDWR | O_CREAT, 0644 | ~07777},
+	{TEST_FILE, 0, ~07777}
+};
+
+static void verify_open(unsigned int n)
+{
+	struct tcase *tc = &tcases[n];
+	struct stat buf;
+
+	TEST(open(tc->filename, tc->flags, tc->mode));
+	fd = TST_RET;
+	if (fd == -1) {
+		tst_res(TFAIL, "Cannot open the file");
+	} else {
+		tst_res(TPASS, "Unknown mode bits were ignored as expected");
+		SAFE_CLOSE(fd);
+	}
+}
+
+static void setup(void)
+{
+}
+
+static void cleanup(void)
+{
+}
+
+static struct tst_test test = {
+	.tcnt = ARRAY_SIZE(tcases),
+	.needs_tmpdir = 1,
+	.setup = setup,
+	.cleanup = cleanup,
+	.test = verify_open,
+};

testcases/kernel/syscalls/openat/.gitignore

@@ -2,3 +2,4 @@
 /openat02
 /openat02_child
 /openat03
+/openat04

testcases/kernel/syscalls/openat/openat04.c

@@ -0,0 +1,67 @@
+/* SPDX-License-Identifier: GPL-2.0-or-later
+ * Copyright (C) 2020 Invisible Things Lab
+ *                    Michał Kowalczyk <[email protected]>
+ */
+
+/*
+ * DESCRIPTION
+ *  Creates a file using `openat` with bits outside of 07777 in `mode` set and
+ *  verifies if they were ignored.
+ *
+ * WARNING
+ *  The fact that these bits are ignored is not documented (at the time of
+ *  writing). Failure of this test doesn't necessarily mean that a regression
+ *  in Linux was introduced, its intention is to catch accidental interface
+ *  changes and warn kernel developers if that happens.
+ */
+
+#include <errno.h>
+#include <fcntl.h>
+#include <sys/stat.h>
+#include <sys/types.h>
+
+#include "tst_test.h"
+
+#define TEST_FILE "testfile"
+
+static int fd;
+
+static struct tcase {
+	char *filename;
+	int flags;
+	mode_t mode;
+} tcases[] = {
+	{TEST_FILE, O_RDWR | O_CREAT, 0644 | ~07777},
+	{TEST_FILE, 0, ~07777}
+};
+
+static void verify_open(unsigned int n)
+{
+	struct tcase *tc = &tcases[n];
+	struct stat buf;
+
+	TEST(openat(AT_FDCWD, tc->filename, tc->flags, tc->mode));
+	fd = TST_RET;
+	if (fd == -1) {
+		tst_res(TFAIL, "Cannot open the file");
+	} else {
+		tst_res(TPASS, "Unknown mode bits were ignored as expected");
+		SAFE_CLOSE(fd);
+	}
+}
+
+static void setup(void)
+{
+}
+
+static void cleanup(void)
+{
+}
+
+static struct tst_test test = {
+	.tcnt = ARRAY_SIZE(tcases),
+	.needs_tmpdir = 1,
+	.setup = setup,
+	.cleanup = cleanup,
+	.test = verify_open,
+};

testcases/kernel/syscalls/openat2/.gitignore

@@ -1,3 +1,4 @@
-openat201
-openat202
-openat203
+/openat201
+/openat202
+/openat203
+/openat204

testcases/kernel/syscalls/openat2/openat204.c

@@ -0,0 +1,68 @@
+/* SPDX-License-Identifier: GPL-2.0-or-later
+ * Copyright (C) 2020 Invisible Things Lab
+ *                    Michał Kowalczyk <[email protected]>
+ */
+
+/*
+ * DESCRIPTION
+ *  Creates a file using `openat2` with bits outside of 07777 in `mode` set and
+ *  verifies if they were ignored.
+ *
+ * WARNING
+ *  The fact that these bits are ignored is not documented (at the time of
+ *  writing). Failure of this test doesn't necessarily mean that a regression
+ *  in Linux was introduced, its intention is to catch accidental interface
+ *  changes and warn kernel developers if that happens.
+ */
+
+#include <errno.h>
+#include <fcntl.h>
+#include <sys/stat.h>
+#include <sys/types.h>
+
+#include "tst_test.h"
+#include "lapi/openat2.h"
+
+#define TEST_FILE "testfile"
+
+static int fd;
+
+static struct tcase {
+	char *filename;
+	int flags;
+	mode_t mode;
+} tcases[] = {
+	{TEST_FILE, O_RDWR | O_CREAT, 0644 | ~07777},
+	{TEST_FILE, 0, ~07777}
+};
+
+static void verify_open(unsigned int n)
+{
+	struct tcase *tc = &tcases[n];
+	struct stat buf;
+
+	TEST(openat2(AT_FDCWD, tc->filename, tc->flags, tc->mode));
+	fd = TST_RET;
+	if (fd == -1) {
+		tst_res(TFAIL, "Cannot open the file");
+	} else {
+		tst_res(TPASS, "Unknown mode bits were ignored as expected");
+		SAFE_CLOSE(fd);
+	}
+}
+
+static void setup(void)
+{
+}
+
+static void cleanup(void)
+{
+}
+
+static struct tst_test test = {
+	.tcnt = ARRAY_SIZE(tcases),
+	.needs_tmpdir = 1,
+	.setup = setup,
+	.cleanup = cleanup,
+	.test = verify_open,
+};