Skip to content

Commit

Permalink
pull more into do_last()
Browse files Browse the repository at this point in the history
Handling of LAST_DOT/LAST_ROOT/LAST_DOTDOT/terminating slash
can be pulled in as well

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
  • Loading branch information
Al Viro committed Mar 5, 2010
1 parent c99658f commit a2c36b4
Showing 1 changed file with 7 additions and 14 deletions.
21 changes: 7 additions & 14 deletions fs/namei.c
Original file line number Diff line number Diff line change
Expand Up @@ -1656,6 +1656,10 @@ static struct file *do_last(struct nameidata *nd, struct path *path,

*is_link = 0;

error = -EISDIR;
if (nd->last_type != LAST_NORM || nd->last.name[nd->last.len])
goto exit;

mutex_lock(&dir->d_inode->i_mutex);

path->dentry = lookup_hash(nd);
Expand Down Expand Up @@ -1826,13 +1830,8 @@ struct file *do_filp_open(int dfd, const char *pathname,
audit_inode(pathname, nd.path.dentry);

/*
* We have the parent and last component. First of all, check
* that we are not asked to creat(2) an obvious directory - that
* will not do.
* We have the parent and last component.
*/
error = -EISDIR;
if (nd.last_type != LAST_NORM || nd.last.name[nd.last.len])
goto exit_parent;

error = -ENFILE;
filp = get_empty_filp();
Expand Down Expand Up @@ -1908,16 +1907,10 @@ struct file *do_filp_open(int dfd, const char *pathname,
nd.flags &= ~LOOKUP_PARENT;
if (nd.last_type == LAST_BIND)
goto ok;
error = -EISDIR;
if (nd.last_type != LAST_NORM)
goto exit;
if (nd.last.name[nd.last.len]) {
__putname(nd.last.name);
goto exit;
}
filp = do_last(&nd, &path, open_flag, flag, acc_mode, mode,
pathname, &is_link);
__putname(nd.last.name);
if (nd.last_type == LAST_NORM)
__putname(nd.last.name);
if (is_link)
goto do_link;
if (nd.root.mnt)
Expand Down

0 comments on commit a2c36b4

Please sign in to comment.