---

yaml --- r: 177393 b: refs/heads/master c: 258fa99 h: refs/heads/master i: 177391: 9502b05 v: v3
git-mirror · Dec 16, 2009 · 65c310d · 65c310d
1 parent 7f56f12
commit 65c310d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/[refs] b/[refs]
@@ -1,2 +1,2 @@
 ---
-refs/heads/master: d231412db66355e72d606f8403ee8b6bd8ad4f9a
+refs/heads/master: 258fa99905f704afed1a43f195bc5235a56fb895
diff --git a/trunk/fs/namei.c b/trunk/fs/namei.c
@@ -644,8 +644,6 @@ static __always_inline int __do_follow_link(struct path *path, struct nameidata
 		if (dentry->d_inode->i_op->put_link)
 			dentry->d_inode->i_op->put_link(dentry, nd, cookie);
 	}
-	path_put(path);
-
 	return error;
 }
 
@@ -672,6 +670,7 @@ static inline int do_follow_link(struct path *path, struct nameidata *nd)
 	current->total_link_count++;
 	nd->depth++;
 	err = __do_follow_link(path, nd);
+	path_put(path);
 	current->link_count--;
 	nd->depth--;
 	return err;
@@ -1864,6 +1863,7 @@ struct file *do_filp_open(int dfd, const char *pathname,
 	if (error)
 		goto exit_dput;
 	error = __do_follow_link(&path, &nd);
+	path_put(&path);
 	if (error) {
 		/* Does someone understand code flow here? Or it is only
 		 * me so stupid? Anathema to whoever designed this non-sense