summaryrefslogtreecommitdiffstats
path: root/fs/namei.c
diff options
context:
space:
mode:
authorAl Viro2011-02-22 21:10:03 +0100
committerAl Viro2011-03-14 14:15:24 +0100
commitfe479a580dc9c737c4eb49ff7fdb31d41d2c7003 (patch)
treecf15923f305d8c4298a44baf2359d2d9e3e2c7c6 /fs/namei.c
parentmerge path_init and path_init_rcu (diff)
downloadkernel-qcow2-linux-fe479a580dc9c737c4eb49ff7fdb31d41d2c7003.tar.gz
kernel-qcow2-linux-fe479a580dc9c737c4eb49ff7fdb31d41d2c7003.tar.xz
kernel-qcow2-linux-fe479a580dc9c737c4eb49ff7fdb31d41d2c7003.zip
merge component type recognition
no need to do it in three places... Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/namei.c')
-rw-r--r--fs/namei.c48
1 files changed, 22 insertions, 26 deletions
diff --git a/fs/namei.c b/fs/namei.c
index b9e537980ef5..4521b5ff7c93 100644
--- a/fs/namei.c
+++ b/fs/namei.c
@@ -1362,6 +1362,7 @@ static int link_path_walk(const char *name, struct nameidata *nd)
unsigned long hash;
struct qstr this;
unsigned int c;
+ int type;
nd->flags |= LOOKUP_CONTINUE;
@@ -1381,6 +1382,16 @@ static int link_path_walk(const char *name, struct nameidata *nd)
this.len = name - (const char *) this.name;
this.hash = end_name_hash(hash);
+ type = LAST_NORM;
+ if (this.name[0] == '.') switch (this.len) {
+ case 2:
+ if (this.name[1] == '.')
+ type = LAST_DOTDOT;
+ break;
+ case 1:
+ type = LAST_DOT;
+ }
+
/* remove trailing slashes? */
if (!c)
goto last_component;
@@ -1393,21 +1404,17 @@ static int link_path_walk(const char *name, struct nameidata *nd)
* to be able to know about the current root directory and
* parent relationships.
*/
- if (this.name[0] == '.') switch (this.len) {
- default:
- break;
- case 2:
- if (this.name[1] != '.')
- break;
+ if (unlikely(type != LAST_NORM)) {
+ if (type == LAST_DOTDOT) {
if (nd->flags & LOOKUP_RCU) {
if (follow_dotdot_rcu(nd))
return -ECHILD;
} else
follow_dotdot(nd);
- /* fallthrough */
- case 1:
- continue;
+ }
+ continue;
}
+
/* This does the actual lookups.. */
err = do_lookup(nd, &this, &next, &inode);
if (err)
@@ -1441,20 +1448,15 @@ last_component:
nd->flags &= lookup_flags | ~LOOKUP_CONTINUE;
if (lookup_flags & LOOKUP_PARENT)
goto lookup_parent;
- if (this.name[0] == '.') switch (this.len) {
- default:
- break;
- case 2:
- if (this.name[1] != '.')
- break;
+ if (unlikely(type != LAST_NORM)) {
+ if (type == LAST_DOTDOT) {
if (nd->flags & LOOKUP_RCU) {
if (follow_dotdot_rcu(nd))
return -ECHILD;
} else
follow_dotdot(nd);
- /* fallthrough */
- case 1:
- goto return_reval;
+ }
+ goto return_reval;
}
err = do_lookup(nd, &this, &next, &inode);
if (err)
@@ -1480,14 +1482,8 @@ last_component:
goto return_base;
lookup_parent:
nd->last = this;
- nd->last_type = LAST_NORM;
- if (this.name[0] != '.')
- goto return_base;
- if (this.len == 1)
- nd->last_type = LAST_DOT;
- else if (this.len == 2 && this.name[1] == '.')
- nd->last_type = LAST_DOTDOT;
- else
+ nd->last_type = type;
+ if (type == LAST_NORM)
goto return_base;
return_reval:
/*