aboutsummaryrefslogtreecommitdiffstats
path: root/fs/namei.c
diff options
context:
space:
mode:
authorAl Viro <viro@zeniv.linux.org.uk>2011-03-16 02:45:02 -0400
committerAl Viro <viro@zeniv.linux.org.uk>2011-03-16 04:57:03 -0400
commit0e794589e588a88d34e339feee50c72606fb21a7 (patch)
tree74c193941b49c0c3bfe3e0934aaf4bbdff22eab4 /fs/namei.c
parentd10902812c9cd5583130a4ebb9ad19c60b68149d (diff)
fix follow_link() breakage
commit 574197e0de46a8a4db5c54ef7b65e43ffa8873a7 had a missing piece, breaking the loop detection ;-/ Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/namei.c')
-rw-r--r--fs/namei.c7
1 files changed, 3 insertions, 4 deletions
diff --git a/fs/namei.c b/fs/namei.c
index 0a601cae23de..b912b7abe747 100644
--- a/fs/namei.c
+++ b/fs/namei.c
@@ -753,9 +753,11 @@ follow_link(struct path *link, struct nameidata *nd, void **p)
753 753
754 BUG_ON(nd->flags & LOOKUP_RCU); 754 BUG_ON(nd->flags & LOOKUP_RCU);
755 755
756 if (link->mnt == nd->path.mnt)
757 mntget(link->mnt);
758
756 if (unlikely(current->total_link_count >= 40)) { 759 if (unlikely(current->total_link_count >= 40)) {
757 *p = ERR_PTR(-ELOOP); /* no ->put_link(), please */ 760 *p = ERR_PTR(-ELOOP); /* no ->put_link(), please */
758 path_put_conditional(link, nd);
759 path_put(&nd->path); 761 path_put(&nd->path);
760 return -ELOOP; 762 return -ELOOP;
761 } 763 }
@@ -765,9 +767,6 @@ follow_link(struct path *link, struct nameidata *nd, void **p)
765 touch_atime(link->mnt, dentry); 767 touch_atime(link->mnt, dentry);
766 nd_set_link(nd, NULL); 768 nd_set_link(nd, NULL);
767 769
768 if (link->mnt == nd->path.mnt)
769 mntget(link->mnt);
770
771 error = security_inode_follow_link(link->dentry, nd); 770 error = security_inode_follow_link(link->dentry, nd);
772 if (error) { 771 if (error) {
773 *p = ERR_PTR(error); /* no ->put_link(), please */ 772 *p = ERR_PTR(error); /* no ->put_link(), please */