On Fri, Nov 25, 2016 at 08:28:47PM +0100, Quentin Casasnovas wrote:
> On Fri, Nov 25, 2016 at 06:09:23PM +0100, Quentin Casasnovas wrote:
> > If two overlayfs filesystems are stacked on top of each other, then we need
> > to recurse when opening a file. This used to work and was first broken by:
> >
> > 4bacc9c9234c ("overlayfs: Make f_path always point to the overlay...")
> >
> > and fixed by:
> >
> > 1c8a47df36d7 ("ovl: fix open in stacked overlay")
> >
> > But it looks like it was re-introduced in:
> >
> > 2d902671ce1c ("vfs: merge .d_select_inode() into .d_real()")
Thanks for the report.
Following patch should fix it (it's there in your patch, so you weren't too far
off). It needs more testing and review, but I think it fixes the basic problem.
Thanks,
Miklos
diff --git a/fs/overlayfs/super.c b/fs/overlayfs/super.c
index edd46a0e951d..f4d2f63fa53a 100644
--- a/fs/overlayfs/super.c
+++ b/fs/overlayfs/super.c
@@ -328,7 +328,7 @@ static struct dentry *ovl_d_real(struct dentry *dentry,
if (!real)
goto bug;
- if (!inode || inode == d_inode(real))
+ if (inode && inode == d_inode(real))
return real;
/* Handle recursion */
