Commit 873f1eed authored by Jeff Layton's avatar Jeff Layton Committed by Al Viro

vfs: turn do_path_lookup into wrapper around struct filename variant

...and make the user_path callers use that variant instead.
Signed-off-by: default avatarJeff Layton <jlayton@redhat.com>
Signed-off-by: default avatarAl Viro <viro@zeniv.linux.org.uk>
parent 7ac86265
...@@ -1988,20 +1988,30 @@ static int path_lookupat(int dfd, const char *name, ...@@ -1988,20 +1988,30 @@ static int path_lookupat(int dfd, const char *name,
return err; return err;
} }
static int do_path_lookup(int dfd, const char *name, static int filename_lookup(int dfd, struct filename *name,
unsigned int flags, struct nameidata *nd) unsigned int flags, struct nameidata *nd)
{ {
int retval = path_lookupat(dfd, name, flags | LOOKUP_RCU, nd); int retval = path_lookupat(dfd, name->name, flags | LOOKUP_RCU, nd);
if (unlikely(retval == -ECHILD)) if (unlikely(retval == -ECHILD))
retval = path_lookupat(dfd, name, flags, nd); retval = path_lookupat(dfd, name->name, flags, nd);
if (unlikely(retval == -ESTALE)) if (unlikely(retval == -ESTALE))
retval = path_lookupat(dfd, name, flags | LOOKUP_REVAL, nd); retval = path_lookupat(dfd, name->name,
flags | LOOKUP_REVAL, nd);
if (likely(!retval)) if (likely(!retval))
audit_inode(name, nd->path.dentry, flags & LOOKUP_PARENT); audit_inode(name->name, nd->path.dentry,
flags & LOOKUP_PARENT);
return retval; return retval;
} }
static int do_path_lookup(int dfd, const char *name,
unsigned int flags, struct nameidata *nd)
{
struct filename filename = { .name = name };
return filename_lookup(dfd, &filename, flags, nd);
}
/* does lookup, returns the object with parent locked */ /* does lookup, returns the object with parent locked */
struct dentry *kern_path_locked(const char *name, struct path *path) struct dentry *kern_path_locked(const char *name, struct path *path)
{ {
...@@ -2125,7 +2135,7 @@ int user_path_at_empty(int dfd, const char __user *name, unsigned flags, ...@@ -2125,7 +2135,7 @@ int user_path_at_empty(int dfd, const char __user *name, unsigned flags,
BUG_ON(flags & LOOKUP_PARENT); BUG_ON(flags & LOOKUP_PARENT);
err = do_path_lookup(dfd, tmp->name, flags, &nd); err = filename_lookup(dfd, tmp, flags, &nd);
putname(tmp); putname(tmp);
if (!err) if (!err)
*path = nd.path; *path = nd.path;
...@@ -2139,6 +2149,12 @@ int user_path_at(int dfd, const char __user *name, unsigned flags, ...@@ -2139,6 +2149,12 @@ int user_path_at(int dfd, const char __user *name, unsigned flags,
return user_path_at_empty(dfd, name, flags, path, NULL); return user_path_at_empty(dfd, name, flags, path, NULL);
} }
/*
* NB: most callers don't do anything directly with the reference to the
* to struct filename, but the nd->last pointer points into the name string
* allocated by getname. So we must hold the reference to it until all
* path-walking is complete.
*/
static struct filename * static struct filename *
user_path_parent(int dfd, const char __user *path, struct nameidata *nd) user_path_parent(int dfd, const char __user *path, struct nameidata *nd)
{ {
...@@ -2148,7 +2164,7 @@ user_path_parent(int dfd, const char __user *path, struct nameidata *nd) ...@@ -2148,7 +2164,7 @@ user_path_parent(int dfd, const char __user *path, struct nameidata *nd)
if (IS_ERR(s)) if (IS_ERR(s))
return s; return s;
error = do_path_lookup(dfd, s->name, LOOKUP_PARENT, nd); error = filename_lookup(dfd, s, LOOKUP_PARENT, nd);
if (error) { if (error) {
putname(s); putname(s);
return ERR_PTR(error); return ERR_PTR(error);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment