aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKirill Korotaev <dev@sw.ru>2006-08-30 22:55:59 +0200
committerAdrian Bunk <bunk@stusta.de>2006-08-30 22:55:59 +0200
commit26c0273b7f4b1510eec4f0f92d19971dc0c2fd67 (patch)
treed3de17af137b40fda19f82eb4198283d43f4d115
parentcbca4f2e2cba4f8d4482d6ddd21199b71c2476b0 (diff)
fix struct file leakage
2.6.16 leaks like hell. While testing, I found massive filp leakage (reproduced in openvz) in the bowels of namei.c. Signed-off-by: Alexey Kuznetsov <kuznet@ms2.inr.ac.ru> Signed-off-by: Adrian Bunk <bunk@stusta.de>
-rw-r--r--fs/namei.c8
1 files changed, 7 insertions, 1 deletions
diff --git a/fs/namei.c b/fs/namei.c
index f11c0aa2293..3ece3a93dd9 100644
--- a/fs/namei.c
+++ b/fs/namei.c
@@ -1708,8 +1708,14 @@ do_link:
if (error)
goto exit_dput;
error = __do_follow_link(&path, nd);
- if (error)
+ if (error) {
+ /* Does someone understand code flow here? Or it is only
+ * me so stupid? Anathema to whoever designed this non-sense
+ * with "intent.open".
+ */
+ release_open_intent(nd);
return error;
+ }
nd->flags &= ~LOOKUP_PARENT;
if (nd->last_type == LAST_BIND)
goto ok;