trailing_symlink: nd->depth massage, part 5
authorAl Viro <viro@zeniv.linux.org.uk>
Mon, 4 May 2015 00:59:58 +0000 (20:59 -0400)
committerAl Viro <viro@zeniv.linux.org.uk>
Mon, 11 May 2015 02:20:04 +0000 (22:20 -0400)
move increment of ->depth to the point where we'd discovered
that get_link() has not returned an error, adjust exits
accordingly.

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
fs/namei.c

index cf5009b87c5a0e49c19a77f932c9192132380f1b..5753f46d83216c3367421b2388cb5190e9e394f7 100644 (file)
@@ -2009,8 +2009,11 @@ static int trailing_symlink(struct nameidata *nd)
        s = get_link(nd);
        if (unlikely(IS_ERR(s)))
                return PTR_ERR(s);
-       if (unlikely(!s))
+       nd->depth++;
+       if (unlikely(!s)) {
+               nd->depth--;
                return 0;
+       }
        if (*s == '/') {
                if (!nd->root.mnt)
                        set_root(nd);
@@ -2020,12 +2023,14 @@ static int trailing_symlink(struct nameidata *nd)
                nd->flags |= LOOKUP_JUMPED;
        }
        nd->inode = nd->path.dentry->d_inode;
-       nd->depth++;
        error = link_path_walk(s, nd);
-       nd->depth--;
-       if (unlikely(error))
+       if (unlikely(error)) {
+               nd->depth--;
                put_link(nd);
-       return error;
+               return error;
+       }
+       nd->depth--;
+       return 0;
 }
 
 static inline int lookup_last(struct nameidata *nd)