summaryrefslogtreecommitdiffstats
path: root/fs/dcache.c
diff options
context:
space:
mode:
authorYan, Zheng <zyan@redhat.com>2014-11-19 15:50:34 +0800
committerAl Viro <viro@zeniv.linux.org.uk>2014-11-19 13:07:20 -0500
commit4a7795d35e252f38298980530e01e21867f8f856 (patch)
tree1ed969cdba05031ff6410a14f32225744b351022 /fs/dcache.c
parentad0eab9293485d1c06237e9249f6d4dfa3d93d4d (diff)
downloadop-kernel-dev-4a7795d35e252f38298980530e01e21867f8f856.zip
op-kernel-dev-4a7795d35e252f38298980530e01e21867f8f856.tar.gz
vfs: fix reference leak in d_prune_aliases()
In "d_prune_alias(): just lock the parent and call __dentry_kill()" the old dget + d_drop + dput has been replaced with lock_parent + __dentry_kill; unfortunately, dput() does more than just killing dentry - it also drops the reference to parent. New variant leaks that reference and needs dput(parent) after killing the child off. Signed-off-by: Yan, Zheng <zyan@redhat.com> Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/dcache.c')
-rw-r--r--fs/dcache.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/fs/dcache.c b/fs/dcache.c
index 3ffef7f..5bc72b0 100644
--- a/fs/dcache.c
+++ b/fs/dcache.c
@@ -778,6 +778,7 @@ restart:
struct dentry *parent = lock_parent(dentry);
if (likely(!dentry->d_lockref.count)) {
__dentry_kill(dentry);
+ dput(parent);
goto restart;
}
if (parent)
OpenPOWER on IntegriCloud