[PATCH 1/1] overlayfs -- ovl_path_open should not take path reference

From: Andy Whitcroft
Date: Thu Jun 27 2013 - 12:27:34 EST


Since the commit below dentry_open now takes its own references
as required. We therefore should no longer take path references in
ovl_path_open. Doing so leaves stray mount references to the underlying
devices preventing them being released:

commit 765927b2d508712d320c8934db963bbe14c3fcec
Author: Al Viro <viro@xxxxxxxxxxxxxxxxxx>
Date: Tue Jun 26 21:58:53 2012 +0400

switch dentry_open() to struct path, make it grab references itself

BugLink: http://bugs.launchpad.net/bugs/1098378
Signed-off-by: Andy Whitcroft <apw@xxxxxxxxxxxxx>
---
fs/overlayfs/super.c | 1 -
1 file changed, 1 deletion(-)


Found this in testing on Ubuntu raring, testing against loopback
mounted files. Without this change we were unable to release the
loopback device for reuse. Looking at it actually we were also leaking
references on the root filesystem, but these are not as obvious.
Applies against overlayfs.v17 as rebased to 3.8 and later.

-apw


diff --git a/fs/overlayfs/super.c b/fs/overlayfs/super.c
index 482c26f..9473e79 100644
--- a/fs/overlayfs/super.c
+++ b/fs/overlayfs/super.c
@@ -385,7 +385,6 @@ struct dentry *ovl_lookup(struct inode *dir, struct dentry *dentry,

struct file *ovl_path_open(struct path *path, int flags)
{
- path_get(path);
return dentry_open(path, flags, current_cred());
}

--
1.8.3.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/