]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commitdiff
ovl: fix regression in fsnotify of overlay merge dir
authorAmir Goldstein <amir73il@gmail.com>
Sun, 28 Jan 2018 00:35:48 +0000 (02:35 +0200)
committerMiklos Szeredi <mszeredi@redhat.com>
Mon, 5 Feb 2018 08:50:29 +0000 (09:50 +0100)
A re-factoring patch in NFS export series has passed the wrong argument
to ovl_get_inode() causing a regression in the very recent fix to
fsnotify of overlay merge dir.

The regression has caused merge directory inodes to be hashed by upper
instead of lower real inode, when NFS export and directory indexing is
disabled. That caused an inotify watch to become obsolete after directory
copy up and drop caches.

LTP test inotify07 was improved to catch this regression.
The regression also caused multiple redirect dirs to same origin not to
be detected on lookup with NFS export disabled. An xfstest was added to
cover this case.

Fixes: 0aceb53e73be ("ovl: do not pass overlay dentry to ovl_get_inode()")
Signed-off-by: Amir Goldstein <amir73il@gmail.com>
Signed-off-by: Miklos Szeredi <mszeredi@redhat.com>
fs/overlayfs/namei.c

index c5449efd96d5e67e4024b50b569925f7e7acfa49..de3e6da1d5a51732f4e0ad3fbad3feb4b6664841 100644 (file)
@@ -976,6 +976,8 @@ struct dentry *ovl_lookup(struct inode *dir, struct dentry *dentry,
                upperdentry = dget(index);
 
        if (upperdentry || ctr) {
+               if (ctr)
+                       origin = stack[0].dentry;
                inode = ovl_get_inode(dentry->d_sb, upperdentry, origin, index,
                                      ctr);
                err = PTR_ERR(inode);