link_path_walk: get rid of duplication
authorAl Viro <viro@zeniv.linux.org.uk>
Sun, 19 Apr 2015 00:14:20 +0000 (20:14 -0400)
committerAl Viro <viro@zeniv.linux.org.uk>
Mon, 11 May 2015 02:19:57 +0000 (22:19 -0400)
What we do after the second walk_component() + put_link() + depth
decrement in there is exactly equivalent to what's done right
after the first walk_component().  Easy to verify and not at all
surprising, seeing that there we have just walked the last
component of nested symlink.

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

index b7ba71889afb9fe26bf55a5b56159e3b04ec5ad6..9f45d339cf4566f2e4ee038e4e3fcff402df8555 100644 (file)
@@ -1771,6 +1771,7 @@ static int link_path_walk(const char *name, struct nameidata *nd)
                        return 0;
 
                err = walk_component(nd, LOOKUP_FOLLOW);
+Walked:
                if (err < 0)
                        return err;
 
@@ -1789,7 +1790,6 @@ static int link_path_walk(const char *name, struct nameidata *nd)
                        nd->depth++;
                        current->link_count++;
 
-loop:  /* will be gone very soon */
                        link = nd->link;
                        s = get_link(&link, nd, &cookie);
 
@@ -1826,13 +1826,7 @@ loop:    /* will be gone very soon */
                                        put_link(nd, &link, cookie);
                                        current->link_count--;
                                        nd->depth--;
-                                       if (err < 0)
-                                               return err;
-                                       if (err > 0) {
-                                               current->link_count++;
-                                               nd->depth++;
-                                               goto loop;
-                                       }
+                                       goto Walked;
                                }
                        }
                }