[CRIU] [PATCH 2/4] zdtm: check read-only bind mounts

Andrey Vagin avagin at openvz.org
Mon Sep 7 12:34:40 PDT 2015


Signed-off-by: Andrey Vagin <avagin at openvz.org>
---
 test/zdtm.sh                        |  3 +-
 test/zdtm/live/static/Makefile      |  1 +
 test/zdtm/live/static/mnt_ro_bind.c | 84 +++++++++++++++++++++++++++++++++++++
 3 files changed, 87 insertions(+), 1 deletion(-)
 create mode 100644 test/zdtm/live/static/mnt_ro_bind.c

diff --git a/test/zdtm.sh b/test/zdtm.sh
index b21ccdd..45ad737 100755
--- a/test/zdtm.sh
+++ b/test/zdtm.sh
@@ -184,6 +184,7 @@ generate_test_list()
 		ns/static/session00
 		ns/static/session01
 		ns/static/tempfs
+		ns/static/mnt_ro_bind
 		ns/static/mount_paths
 		ns/static/bind-mount
 		static/utsname
@@ -323,9 +324,9 @@ chroot-file
 console
 vt
 rtc
-tempfs
 maps007
 tempfs
+mnt_ro_bind
 bind-mount
 mountpoints
 inotify_irmap
diff --git a/test/zdtm/live/static/Makefile b/test/zdtm/live/static/Makefile
index 67be675..755d7ad 100644
--- a/test/zdtm/live/static/Makefile
+++ b/test/zdtm/live/static/Makefile
@@ -182,6 +182,7 @@ TST_DIR		=				\
 		overmount_fifo			\
 		overmount_sock			\
 		tempfs				\
+		mnt_ro_bind			\
 		mount_paths			\
 		bind-mount			\
 		inotify00			\
diff --git a/test/zdtm/live/static/mnt_ro_bind.c b/test/zdtm/live/static/mnt_ro_bind.c
new file mode 100644
index 0000000..1d98814
--- /dev/null
+++ b/test/zdtm/live/static/mnt_ro_bind.c
@@ -0,0 +1,84 @@
+#include <stdbool.h>
+#include <string.h>
+#include <fcntl.h>
+#include <unistd.h>
+#include <sys/mount.h>
+#include <sys/stat.h>
+#include <linux/limits.h>
+#include <errno.h>
+
+#include "zdtmtst.h"
+
+const char *test_doc	= "Check read-only bind-mounts";
+const char *test_author	= "Andrew Vagin <avagin at openvz.org>";
+
+char *dirname;
+TEST_OPTION(dirname, string, "directory name", 1);
+
+#define TEST_WORD	"testtest"
+#define TEST_WORD2	"TESTTEST"
+
+int main(int argc, char **argv)
+{
+	int fd, ret = 1;
+	char rw_path[PATH_MAX], ro_path[PATH_MAX], rw_f[PATH_MAX], ro_f[PATH_MAX];
+
+	test_init(argc, argv);
+
+	snprintf(rw_path, sizeof(rw_path), "%s/rw", dirname);
+	snprintf(ro_path, sizeof(ro_path), "%s/ro", dirname);
+	snprintf(rw_f, sizeof(rw_f), "%s/rw/test", dirname);
+	snprintf(ro_f, sizeof(ro_f), "%s/ro/test", dirname);
+
+	mkdir(dirname, 0700);
+	if (mount("none", dirname, "tmpfs", 0, "") < 0) {
+		fail("Can't mount tmpfs");
+		return 1;
+	}
+	mkdir(rw_path, 0700);
+	mkdir(ro_path, 0700);
+
+	if (mount("zdtm_rw", rw_path, "tmpfs", 0, "") < 0) {
+		fail("Can't mount tmpfs");
+		return 1;
+	}
+
+	if (mount(rw_path, ro_path, NULL, MS_BIND, NULL) < 0) {
+		fail("Can't mount tmpfs");
+		return 1;
+	}
+
+	if (mount(NULL, ro_path, NULL, MS_BIND | MS_REMOUNT | MS_RDONLY, NULL) < 0) {
+		fail("Can't mount tmpfs");
+		return 1;
+	}
+
+	test_daemon();
+	test_waitsig();
+
+	fd = open(ro_f, O_CREAT | O_WRONLY, 0666);
+	if (fd != -1 || errno != EROFS) {
+		fail("%s is created", ro_f);
+		goto err;
+	}
+
+	fd = open(rw_f, O_CREAT | O_WRONLY, 0666);
+	if (fd < 0) {
+		fail("Unable to create %s", rw_f);
+		goto err;
+	}
+	close(fd);
+
+	fd = open(ro_f, O_RDONLY);
+	if (fd < 0) {
+		fail("Unable to create %s", rw_f);
+		goto err;
+	}
+
+	pass();
+	ret = 0;
+err:
+	umount2(dirname, MNT_DETACH);
+	rmdir(dirname);
+	return ret;
+}
-- 
2.4.3



More information about the CRIU mailing list