fanotify: report old and/or new parent+name in FAN_RENAME event

In the special case of FAN_RENAME event, we report old or new or both
old and new parent+name.

A single info record will be reported if either the old or new dir
is watched and two records will be reported if both old and new dir
(or their filesystem) are watched.

The old and new parent+name are reported using new info record types
FAN_EVENT_INFO_TYPE_{OLD,NEW}_DFID_NAME, so if a single info record
is reported, it is clear to the application, to which dir entry the
fid+name info is referring to.

Link: https://lore.kernel.org/r/20211129201537.1932819-11-amir73il@gmail.com
Signed-off-by: Amir Goldstein <amir73il@gmail.com>
Signed-off-by: Jan Kara <jack@suse.cz>

diff --git a/fs/notify/fanotify/fanotify.c b/fs/notify/fanotify/fanotify.c
index 14bc0f12cc9f31f413d40af52e3bb1aaa9fe27f3..0da305b6f3e2f5e69d53bd33022aaee91a08f8a2 100644 (file)
--- a/fs/notify/fanotify/fanotify.c
+++ b/fs/notify/fanotify/fanotify.c
@@ -153,6 +153,13 @@ static bool fanotify_should_merge(struct fanotify_event *old,
        if ((old->mask & FS_ISDIR) != (new->mask & FS_ISDIR))
                return false;
 
+       /*
+        * FAN_RENAME event is reported with special info record types,
+        * so we cannot merge it with other events.
+        */
+       if ((old->mask & FAN_RENAME) != (new->mask & FAN_RENAME))
+               return false;
+
        switch (old->type) {
        case FANOTIFY_EVENT_TYPE_PATH:
                return fanotify_path_equal(fanotify_event_path(old),

diff --git a/fs/notify/fanotify/fanotify.h b/fs/notify/fanotify/fanotify.h
index 8fa3bc0effd4531e205e6505c91ceda0591431a8..a3d5b751cac5b775eb28d4d5ca0fc5f5a267e374 100644 (file)
--- a/fs/notify/fanotify/fanotify.h
+++ b/fs/notify/fanotify/fanotify.h
@@ -373,6 +373,13 @@ static inline int fanotify_event_dir_fh_len(struct fanotify_event *event)
        return info ? fanotify_info_dir_fh_len(info) : 0;
 }
 
+static inline int fanotify_event_dir2_fh_len(struct fanotify_event *event)
+{
+       struct fanotify_info *info = fanotify_event_info(event);
+
+       return info ? fanotify_info_dir2_fh_len(info) : 0;
+}
+
 static inline bool fanotify_event_has_object_fh(struct fanotify_event *event)
 {
        /* For error events, even zeroed fh are reported. */
@@ -386,6 +393,17 @@ static inline bool fanotify_event_has_dir_fh(struct fanotify_event *event)
        return fanotify_event_dir_fh_len(event) > 0;
 }
 
+static inline bool fanotify_event_has_dir2_fh(struct fanotify_event *event)
+{
+       return fanotify_event_dir2_fh_len(event) > 0;
+}
+
+static inline bool fanotify_event_has_any_dir_fh(struct fanotify_event *event)
+{
+       return fanotify_event_has_dir_fh(event) ||
+               fanotify_event_has_dir2_fh(event);
+}
+
 struct fanotify_path_event {
        struct fanotify_event fae;
        struct path path;

diff --git a/fs/notify/fanotify/fanotify_user.c b/fs/notify/fanotify/fanotify_user.c
index e4a11f56782d52fb14fa7bf80dab23500f7435b8..eb2a0251b3188f429f900c5fc980b10ee29a79ea 100644 (file)
--- a/fs/notify/fanotify/fanotify_user.c
+++ b/fs/notify/fanotify/fanotify_user.c
@@ -129,12 +129,29 @@ static int fanotify_fid_info_len(int fh_len, int name_len)
                       FANOTIFY_EVENT_ALIGN);
 }
 
+/* FAN_RENAME may have one or two dir+name info records */
+static int fanotify_dir_name_info_len(struct fanotify_event *event)
+{
+       struct fanotify_info *info = fanotify_event_info(event);
+       int dir_fh_len = fanotify_event_dir_fh_len(event);
+       int dir2_fh_len = fanotify_event_dir2_fh_len(event);
+       int info_len = 0;
+
+       if (dir_fh_len)
+               info_len += fanotify_fid_info_len(dir_fh_len,
+                                                 info->name_len);
+       if (dir2_fh_len)
+               info_len += fanotify_fid_info_len(dir2_fh_len,
+                                                 info->name2_len);
+
+       return info_len;
+}
+
 static size_t fanotify_event_len(unsigned int info_mode,
                                 struct fanotify_event *event)
 {
        size_t event_len = FAN_EVENT_METADATA_LEN;
        struct fanotify_info *info;
-       int dir_fh_len;
        int fh_len;
        int dot_len = 0;
 
@@ -146,9 +163,8 @@ static size_t fanotify_event_len(unsigned int info_mode,
 
        info = fanotify_event_info(event);
 
-       if (fanotify_event_has_dir_fh(event)) {
-               dir_fh_len = fanotify_event_dir_fh_len(event);
-               event_len += fanotify_fid_info_len(dir_fh_len, info->name_len);
+       if (fanotify_event_has_any_dir_fh(event)) {
+               event_len += fanotify_dir_name_info_len(event);
        } else if ((info_mode & FAN_REPORT_NAME) &&
                   (event->mask & FAN_ONDIR)) {
                /*
@@ -379,6 +395,8 @@ static int copy_fid_info_to_user(__kernel_fsid_t *fsid, struct fanotify_fh *fh,
                        return -EFAULT;
                break;
        case FAN_EVENT_INFO_TYPE_DFID_NAME:
+       case FAN_EVENT_INFO_TYPE_OLD_DFID_NAME:
+       case FAN_EVENT_INFO_TYPE_NEW_DFID_NAME:
                if (WARN_ON_ONCE(!name || !name_len))
                        return -EFAULT;
                break;
@@ -478,11 +496,19 @@ static int copy_info_records_to_user(struct fanotify_event *event,
        unsigned int pidfd_mode = info_mode & FAN_REPORT_PIDFD;
 
        /*
-        * Event info records order is as follows: dir fid + name, child fid.
+        * Event info records order is as follows:
+        * 1. dir fid + name
+        * 2. (optional) new dir fid + new name
+        * 3. (optional) child fid
         */
        if (fanotify_event_has_dir_fh(event)) {
                info_type = info->name_len ? FAN_EVENT_INFO_TYPE_DFID_NAME :
                                             FAN_EVENT_INFO_TYPE_DFID;
+
+               /* FAN_RENAME uses special info types */
+               if (event->mask & FAN_RENAME)
+                       info_type = FAN_EVENT_INFO_TYPE_OLD_DFID_NAME;
+
                ret = copy_fid_info_to_user(fanotify_event_fsid(event),
                                            fanotify_info_dir_fh(info),
                                            info_type,
@@ -496,6 +522,22 @@ static int copy_info_records_to_user(struct fanotify_event *event,
                total_bytes += ret;
        }
 
+       /* New dir fid+name may be reported in addition to old dir fid+name */
+       if (fanotify_event_has_dir2_fh(event)) {
+               info_type = FAN_EVENT_INFO_TYPE_NEW_DFID_NAME;
+               ret = copy_fid_info_to_user(fanotify_event_fsid(event),
+                                           fanotify_info_dir2_fh(info),
+                                           info_type,
+                                           fanotify_info_name2(info),
+                                           info->name2_len, buf, count);
+               if (ret < 0)
+                       return ret;
+
+               buf += ret;
+               count -= ret;
+               total_bytes += ret;
+       }
+
        if (fanotify_event_has_object_fh(event)) {
                const char *dot = NULL;
                int dot_len = 0;

diff --git a/include/uapi/linux/fanotify.h b/include/uapi/linux/fanotify.h
index 9d0e2dc5767b50dbcca907136844536168af9aad..e8ac38cc2fd6d8f64dbedac5ecf83cefef3c645e 100644 (file)
--- a/include/uapi/linux/fanotify.h
+++ b/include/uapi/linux/fanotify.h
@@ -134,6 +134,12 @@ struct fanotify_event_metadata {
 #define FAN_EVENT_INFO_TYPE_PIDFD      4
 #define FAN_EVENT_INFO_TYPE_ERROR      5
 
+/* Special info types for FAN_RENAME */
+#define FAN_EVENT_INFO_TYPE_OLD_DFID_NAME      10
+/* Reserved for FAN_EVENT_INFO_TYPE_OLD_DFID   11 */
+#define FAN_EVENT_INFO_TYPE_NEW_DFID_NAME      12
+/* Reserved for FAN_EVENT_INFO_TYPE_NEW_DFID   13 */
+
 /* Variable length info record following event metadata */
 struct fanotify_event_info_header {
        __u8 info_type;