From mboxrd@z Thu Jan 1 00:00:00 1970 From: Amir Goldstein Date: Fri, 28 Sep 2018 10:51:17 +0300 Subject: [LTP] [PATCH v3] syscalls/fanotify10: new test for mount ignore mask Message-ID: <20180928075117.4789-1-amir73il@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: ltp@lists.linux.it This is a regression test for commit: 9bdda4e9cf2d fsnotify: fix ignore mask logic in fsnotify() The test is derived from fanotify06 and adds more test cases. Signed-off-by: Amir Goldstein --- Cyril, Please accept this regression test for a bug that was fixes upstream. Thanks, Amir. Changes from v2: - Use SPDX identifier Changes from v1: - Use test index to iterate test cases runtest/syscalls | 1 + testcases/kernel/syscalls/fanotify/.gitignore | 1 + testcases/kernel/syscalls/fanotify/fanotify.h | 12 + .../kernel/syscalls/fanotify/fanotify10.c | 313 ++++++++++++++++++ 4 files changed, 327 insertions(+) create mode 100644 testcases/kernel/syscalls/fanotify/fanotify10.c diff --git a/runtest/syscalls b/runtest/syscalls index 0d0be7713..cd52561a5 100644 --- a/runtest/syscalls +++ b/runtest/syscalls @@ -497,6 +497,7 @@ fanotify06 fanotify06 fanotify07 fanotify07 fanotify08 fanotify08 fanotify09 fanotify09 +fanotify10 fanotify10 ioperm01 ioperm01 ioperm02 ioperm02 diff --git a/testcases/kernel/syscalls/fanotify/.gitignore b/testcases/kernel/syscalls/fanotify/.gitignore index ec7553995..c26f2bd27 100644 --- a/testcases/kernel/syscalls/fanotify/.gitignore +++ b/testcases/kernel/syscalls/fanotify/.gitignore @@ -7,3 +7,4 @@ /fanotify07 /fanotify08 /fanotify09 +/fanotify10 diff --git a/testcases/kernel/syscalls/fanotify/fanotify.h b/testcases/kernel/syscalls/fanotify/fanotify.h index f5f7df25e..5adef54d7 100644 --- a/testcases/kernel/syscalls/fanotify/fanotify.h +++ b/testcases/kernel/syscalls/fanotify/fanotify.h @@ -54,4 +54,16 @@ static long fanotify_mark(int fd, unsigned int flags, uint64_t mask, #endif /* HAVE_SYS_FANOTIFY_H */ +#ifndef FAN_MARK_INODE +#define FAN_MARK_INODE 0 +#endif + +struct fanotify_mark_type { + unsigned int flag; + const char * name; +}; + +#define INIT_FANOTIFY_MARK_TYPE(t) \ + { FAN_MARK_ ## t, "FAN_MARK_" #t } + #endif /* __FANOTIFY_H__ */ diff --git a/testcases/kernel/syscalls/fanotify/fanotify10.c b/testcases/kernel/syscalls/fanotify/fanotify10.c new file mode 100644 index 000000000..e71264be2 --- /dev/null +++ b/testcases/kernel/syscalls/fanotify/fanotify10.c @@ -0,0 +1,313 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Copyright (c) 2014 SUSE. All Rights Reserved. + * Copyright (c) 2018 CTERA Networks. All Rights Reserved. + * + * Started by Jan Kara + * Forked from fanotify06.c by Amir Goldstein + * + * DESCRIPTION + * Check that fanotify properly merges ignore mask of a mount mark + * with a mask of an inode mark on the same group. Unlike the + * prototype test fanotify06, do not use FAN_MODIFY event for the + * test mask, because it hides the bug. + * + * This is a regression test for commit: + * + * 9bdda4e9cf2d fsnotify: fix ignore mask logic in fsnotify() + */ +#define _GNU_SOURCE +#include "config.h" + +#include +#include +#include +#include +#include +#include +#include +#include +#include "tst_test.h" +#include "fanotify.h" + +#if defined(HAVE_SYS_FANOTIFY_H) +#include + +#define EVENT_MAX 1024 +/* size of the event structure, not counting name */ +#define EVENT_SIZE (sizeof (struct fanotify_event_metadata)) +/* reasonable guess as to size of 1024 events */ +#define EVENT_BUF_LEN (EVENT_MAX * EVENT_SIZE) + +static unsigned int fanotify_prio[] = { + FAN_CLASS_PRE_CONTENT, + FAN_CLASS_CONTENT, + FAN_CLASS_NOTIF +}; +#define FANOTIFY_PRIORITIES ARRAY_SIZE(fanotify_prio) + +#define GROUPS_PER_PRIO 3 + +static int fd_notify[FANOTIFY_PRIORITIES][GROUPS_PER_PRIO]; + +static char event_buf[EVENT_BUF_LEN]; + +#define MOUNT_PATH "fs_mnt" +#define MNT2_PATH "mntpoint" +#define FILE_NAME "testfile" +#define FILE2_NAME "testfile2" +#define FILE_PATH MOUNT_PATH"/"FILE_NAME +#define FILE2_PATH MOUNT_PATH"/"FILE2_NAME +#define FILE_MNT2 MNT2_PATH"/"FILE_NAME +#define FILE2_MNT2 MNT2_PATH"/"FILE2_NAME + +static int mount_created, bind_mount_created; + +enum { + FANOTIFY_INODE, + FANOTIFY_MOUNT, +}; + +static struct fanotify_mark_type fanotify_mark_types[] = { + INIT_FANOTIFY_MARK_TYPE(INODE), + INIT_FANOTIFY_MARK_TYPE(MOUNT), +}; + +static struct tcase { + const char *tname; + const char *mark_path; + int mark_type; + const char *ignore_path; + int ignore_mark_type; + const char *event_path; + int expect_event; +} tcases[] = { + { + "ignore mount events created on a specific file", + MOUNT_PATH, FANOTIFY_MOUNT, + FILE_MNT2, FANOTIFY_INODE, + FILE_PATH, 0 + }, + { + "don't ignore mount events created on another file", + MOUNT_PATH, FANOTIFY_MOUNT, + FILE_PATH, FANOTIFY_INODE, + FILE2_PATH, 1 + }, + { + "ignore inode events created on a specific mount point", + FILE_PATH, FANOTIFY_INODE, + MNT2_PATH, FANOTIFY_MOUNT, + FILE_MNT2, 0 + }, + { + "don't ignore inode events created on another mount point", + FILE_MNT2, FANOTIFY_INODE, + MNT2_PATH, FANOTIFY_MOUNT, + FILE_PATH, 1 + }, +}; + +static void create_fanotify_groups(unsigned int n) +{ + struct tcase *tc = &tcases[n]; + struct fanotify_mark_type *mark, *ignore_mark; + unsigned int p, i; + int ret; + + mark = &fanotify_mark_types[tc->mark_type]; + ignore_mark = &fanotify_mark_types[tc->ignore_mark_type]; + + for (p = 0; p < FANOTIFY_PRIORITIES; p++) { + for (i = 0; i < GROUPS_PER_PRIO; i++) { + fd_notify[p][i] = SAFE_FANOTIFY_INIT(fanotify_prio[p] | + FAN_NONBLOCK, + O_RDONLY); + + /* Add mark for each group */ + ret = fanotify_mark(fd_notify[p][i], + FAN_MARK_ADD | mark->flag, + FAN_OPEN, AT_FDCWD, tc->mark_path); + if (ret < 0) { + tst_brk(TBROK | TERRNO, + "fanotify_mark(%d, FAN_MARK_ADD | %s," + "FAN_OPEN, AT_FDCWD, %s) failed", + fd_notify[p][i], mark->name, + tc->mark_path); + } + /* Add ignore mark for groups with higher priority */ + if (p == 0) + continue; + ret = fanotify_mark(fd_notify[p][i], + FAN_MARK_ADD | ignore_mark->flag | + FAN_MARK_IGNORED_MASK | + FAN_MARK_IGNORED_SURV_MODIFY, + FAN_OPEN, AT_FDCWD, + tc->ignore_path); + if (ret < 0) { + tst_brk(TBROK | TERRNO, + "fanotify_mark(%d, FAN_MARK_ADD | %s | " + "FAN_MARK_IGNORED_MASK | " + "FAN_MARK_IGNORED_SURV_MODIFY, " + "FAN_OPEN, AT_FDCWD, %s) failed", + fd_notify[p][i], ignore_mark->name, + tc->ignore_path); + } + } + } +} + +static void cleanup_fanotify_groups(void) +{ + unsigned int i, p; + + for (p = 0; p < FANOTIFY_PRIORITIES; p++) { + for (i = 0; i < GROUPS_PER_PRIO; i++) { + if (fd_notify[p][i] > 0) + SAFE_CLOSE(fd_notify[p][i]); + } + } +} + +static void verify_event(int group, struct fanotify_event_metadata *event) +{ + if (event->mask != FAN_OPEN) { + tst_res(TFAIL, "group %d got event: mask %llx (expected %llx) " + "pid=%u fd=%u", group, (unsigned long long)event->mask, + (unsigned long long)FAN_OPEN, + (unsigned)event->pid, event->fd); + } else if (event->pid != getpid()) { + tst_res(TFAIL, "group %d got event: mask %llx pid=%u " + "(expected %u) fd=%u", group, + (unsigned long long)event->mask, (unsigned)event->pid, + (unsigned)getpid(), event->fd); + } else { + tst_res(TPASS, "group %d got event: mask %llx pid=%u fd=%u", + group, (unsigned long long)event->mask, + (unsigned)event->pid, event->fd); + } +} + +static void test_fanotify(unsigned int n) +{ + struct tcase *tc = &tcases[n]; + struct fanotify_mark_type *mark, *ignore_mark; + int ret, fd; + unsigned int p, i; + struct fanotify_event_metadata *event; + + tst_res(TINFO, "Test #%d: %s", n, tc->tname); + + create_fanotify_groups(n); + + mark = &fanotify_mark_types[tc->mark_type]; + ignore_mark = &fanotify_mark_types[tc->ignore_mark_type]; + + /* + * generate sequence of events + */ + fd = SAFE_OPEN(tc->event_path, O_RDONLY); + SAFE_CLOSE(fd); + + /* First verify all groups without matching ignore mask got the event */ + for (p = 0; p < FANOTIFY_PRIORITIES; p++) { + if (p > 0 && !tc->expect_event) + break; + + for (i = 0; i < GROUPS_PER_PRIO; i++) { + ret = read(fd_notify[p][i], event_buf, EVENT_BUF_LEN); + if (ret < 0) { + if (errno == EAGAIN) { + tst_res(TFAIL, "group %d (prio %d) " + "with %s did not get event", + i, p, mark->name); + } + tst_brk(TBROK | TERRNO, + "reading fanotify events failed"); + } + if (ret < (int)FAN_EVENT_METADATA_LEN) { + tst_brk(TBROK, + "short read when reading fanotify " + "events (%d < %d)", ret, + (int)EVENT_BUF_LEN); + } + event = (struct fanotify_event_metadata *)event_buf; + if (ret > (int)event->event_len) { + tst_res(TFAIL, "group %d (prio %d) with %s " + "got more than one event (%d > %d)", + i, p, mark->name, ret, + event->event_len); + } else { + verify_event(i, event); + } + if (event->fd != FAN_NOFD) + SAFE_CLOSE(event->fd); + } + } + /* Then verify all groups with matching ignore mask did got the event */ + for (p = 1; p < FANOTIFY_PRIORITIES && !tc->expect_event; p++) { + for (i = 0; i < GROUPS_PER_PRIO; i++) { + ret = read(fd_notify[p][i], event_buf, EVENT_BUF_LEN); + if (ret == 0) { + tst_brk(TBROK, + "zero length read from fanotify fd"); + } + if (ret > 0) { + tst_res(TFAIL, "group %d (prio %d) with %s and " + "%s ignore mask got event", + i, p, mark->name, ignore_mark->name); + if (event->fd != FAN_NOFD) + SAFE_CLOSE(event->fd); + } else if (errno == EAGAIN) { + tst_res(TPASS, "group %d (prio %d) with %s and " + "%s ignore mask got no event", + i, p, mark->name, ignore_mark->name); + } else { + tst_brk(TBROK | TERRNO, + "reading fanotify events failed"); + } + } + } + cleanup_fanotify_groups(); +} + +static void setup(void) +{ + /* Mount the filesystem at one path for watching events */ + SAFE_MKDIR(MOUNT_PATH, 0755); + SAFE_MOUNT(tst_device->dev, MOUNT_PATH, tst_device->fs_type, 0, NULL); + mount_created = 1; + + /* Create another bind mount at another path for generating events */ + SAFE_MKDIR(MNT2_PATH, 0755); + SAFE_MOUNT(MOUNT_PATH, MNT2_PATH, "none", MS_BIND, NULL); + bind_mount_created = 1; + + SAFE_FILE_PRINTF(FILE_PATH, "1"); + SAFE_FILE_PRINTF(FILE2_PATH, "1"); +} + +static void cleanup(void) +{ + cleanup_fanotify_groups(); + + if (bind_mount_created && tst_umount(MNT2_PATH) < 0) + tst_brk(TBROK | TERRNO, "bind umount failed"); + + if (mount_created && tst_umount(MOUNT_PATH) < 0) + tst_brk(TBROK | TERRNO, "fs umount failed"); +} + +static struct tst_test test = { + .test = test_fanotify, + .tcnt = ARRAY_SIZE(tcases), + .setup = setup, + .cleanup = cleanup, + .format_device = 1, + .needs_tmpdir = 1, + .needs_root = 1 +}; + +#else + TST_TEST_TCONF("system doesn't have required fanotify support"); +#endif -- 2.17.1