audit: fix filename matching in __audit_inode() and __audit_inode_child()
[linux-2.6-block.git] / fs / namei.c
index ca814165d84cc25214740eb72b81c4d3d6c07b46..a3fde77d4abffd0270a53d06bcd22bc3d25a1558 100644 (file)
@@ -212,32 +212,39 @@ getname(const char __user * filename)
        return getname_flags(filename, 0, NULL);
 }
 
-/*
- * The "getname_kernel()" interface doesn't do pathnames longer
- * than EMBEDDED_NAME_MAX. Deal with it - you're a kernel user.
- */
 struct filename *
 getname_kernel(const char * filename)
 {
        struct filename *result;
-       char *kname;
-       int len;
-
-       len = strlen(filename);
-       if (len >= EMBEDDED_NAME_MAX)
-               return ERR_PTR(-ENAMETOOLONG);
+       int len = strlen(filename) + 1;
 
        result = __getname();
        if (unlikely(!result))
                return ERR_PTR(-ENOMEM);
 
-       kname = (char *)result + sizeof(*result);
-       result->name = kname;
+       if (len <= EMBEDDED_NAME_MAX) {
+               result->name = (char *)(result) + sizeof(*result);
+               result->separate = false;
+       } else if (len <= PATH_MAX) {
+               struct filename *tmp;
+
+               tmp = kmalloc(sizeof(*tmp), GFP_KERNEL);
+               if (unlikely(!tmp)) {
+                       __putname(result);
+                       return ERR_PTR(-ENOMEM);
+               }
+               tmp->name = (char *)result;
+               tmp->separate = true;
+               result = tmp;
+       } else {
+               __putname(result);
+               return ERR_PTR(-ENAMETOOLONG);
+       }
+       memcpy((char *)result->name, filename, len);
        result->uptr = NULL;
        result->aname = NULL;
-       result->separate = false;
+       audit_getname(result);
 
-       strlcpy(kname, filename, EMBEDDED_NAME_MAX);
        return result;
 }
 
@@ -487,6 +494,19 @@ void path_put(const struct path *path)
 }
 EXPORT_SYMBOL(path_put);
 
+struct nameidata {
+       struct path     path;
+       struct qstr     last;
+       struct path     root;
+       struct inode    *inode; /* path.dentry.d_inode */
+       unsigned int    flags;
+       unsigned        seq, m_seq;
+       int             last_type;
+       unsigned        depth;
+       struct file     *base;
+       char *saved_names[MAX_NESTED_LINKS + 1];
+};
+
 /*
  * Path walking has 2 modes, rcu-walk and ref-walk (see
  * Documentation/filesystems/path-lookup.txt).  In situations when we can't
@@ -695,6 +715,18 @@ void nd_jump_link(struct nameidata *nd, struct path *path)
        nd->flags |= LOOKUP_JUMPED;
 }
 
+void nd_set_link(struct nameidata *nd, char *path)
+{
+       nd->saved_names[nd->depth] = path;
+}
+EXPORT_SYMBOL(nd_set_link);
+
+char *nd_get_link(struct nameidata *nd)
+{
+       return nd->saved_names[nd->depth];
+}
+EXPORT_SYMBOL(nd_get_link);
+
 static inline void put_link(struct nameidata *nd, struct path *link, void *cookie)
 {
        struct inode *inode = link->dentry->d_inode;
@@ -1821,13 +1853,14 @@ static int link_path_walk(const char *name, struct nameidata *nd)
 }
 
 static int path_init(int dfd, const char *name, unsigned int flags,
-                    struct nameidata *nd, struct file **fp)
+                    struct nameidata *nd)
 {
        int retval = 0;
 
        nd->last_type = LAST_ROOT; /* if there are only slashes... */
-       nd->flags = flags | LOOKUP_JUMPED;
+       nd->flags = flags | LOOKUP_JUMPED | LOOKUP_PARENT;
        nd->depth = 0;
+       nd->base = NULL;
        if (flags & LOOKUP_ROOT) {
                struct dentry *root = nd->root.dentry;
                struct inode *inode = root->d_inode;
@@ -1847,7 +1880,7 @@ static int path_init(int dfd, const char *name, unsigned int flags,
                } else {
                        path_get(&nd->path);
                }
-               return 0;
+               goto done;
        }
 
        nd->root.mnt = NULL;
@@ -1897,7 +1930,7 @@ static int path_init(int dfd, const char *name, unsigned int flags,
                nd->path = f.file->f_path;
                if (flags & LOOKUP_RCU) {
                        if (f.flags & FDPUT_FPUT)
-                               *fp = f.file;
+                               nd->base = f.file;
                        nd->seq = __read_seqcount_begin(&nd->path.dentry->d_seq);
                        rcu_read_lock();
                } else {
@@ -1908,13 +1941,26 @@ static int path_init(int dfd, const char *name, unsigned int flags,
 
        nd->inode = nd->path.dentry->d_inode;
        if (!(flags & LOOKUP_RCU))
-               return 0;
+               goto done;
        if (likely(!read_seqcount_retry(&nd->path.dentry->d_seq, nd->seq)))
-               return 0;
+               goto done;
        if (!(nd->flags & LOOKUP_ROOT))
                nd->root.mnt = NULL;
        rcu_read_unlock();
        return -ECHILD;
+done:
+       current->total_link_count = 0;
+       return link_path_walk(name, nd);
+}
+
+static void path_cleanup(struct nameidata *nd)
+{
+       if (nd->root.mnt && !(nd->flags & LOOKUP_ROOT)) {
+               path_put(&nd->root);
+               nd->root.mnt = NULL;
+       }
+       if (unlikely(nd->base))
+               fput(nd->base);
 }
 
 static inline int lookup_last(struct nameidata *nd, struct path *path)
@@ -1930,7 +1976,6 @@ static inline int lookup_last(struct nameidata *nd, struct path *path)
 static int path_lookupat(int dfd, const char *name,
                                unsigned int flags, struct nameidata *nd)
 {
-       struct file *base = NULL;
        struct path path;
        int err;
 
@@ -1948,14 +1993,7 @@ static int path_lookupat(int dfd, const char *name,
         * be handled by restarting a traditional ref-walk (which will always
         * be able to complete).
         */
-       err = path_init(dfd, name, flags | LOOKUP_PARENT, nd, &base);
-
-       if (unlikely(err))
-               goto out;
-
-       current->total_link_count = 0;
-       err = link_path_walk(name, nd);
-
+       err = path_init(dfd, name, flags, nd);
        if (!err && !(flags & LOOKUP_PARENT)) {
                err = lookup_last(nd, &path);
                while (err > 0) {
@@ -1983,14 +2021,7 @@ static int path_lookupat(int dfd, const char *name,
                }
        }
 
-out:
-       if (base)
-               fput(base);
-
-       if (nd->root.mnt && !(nd->flags & LOOKUP_ROOT)) {
-               path_put(&nd->root);
-               nd->root.mnt = NULL;
-       }
+       path_cleanup(nd);
        return err;
 }
 
@@ -2012,31 +2043,47 @@ static int filename_lookup(int dfd, struct filename *name,
 static int do_path_lookup(int dfd, const char *name,
                                unsigned int flags, struct nameidata *nd)
 {
-       struct filename filename = { .name = name };
+       struct filename *filename = getname_kernel(name);
+       int retval = PTR_ERR(filename);
 
-       return filename_lookup(dfd, &filename, flags, nd);
+       if (!IS_ERR(filename)) {
+               retval = filename_lookup(dfd, filename, flags, nd);
+               putname(filename);
+       }
+       return retval;
 }
 
 /* does lookup, returns the object with parent locked */
 struct dentry *kern_path_locked(const char *name, struct path *path)
 {
+       struct filename *filename = getname_kernel(name);
        struct nameidata nd;
        struct dentry *d;
-       int err = do_path_lookup(AT_FDCWD, name, LOOKUP_PARENT, &nd);
-       if (err)
-               return ERR_PTR(err);
+       int err;
+
+       if (IS_ERR(filename))
+               return ERR_CAST(filename);
+
+       err = filename_lookup(AT_FDCWD, filename, LOOKUP_PARENT, &nd);
+       if (err) {
+               d = ERR_PTR(err);
+               goto out;
+       }
        if (nd.last_type != LAST_NORM) {
                path_put(&nd.path);
-               return ERR_PTR(-EINVAL);
+               d = ERR_PTR(-EINVAL);
+               goto out;
        }
        mutex_lock_nested(&nd.path.dentry->d_inode->i_mutex, I_MUTEX_PARENT);
        d = __lookup_hash(&nd.last, nd.path.dentry, 0);
        if (IS_ERR(d)) {
                mutex_unlock(&nd.path.dentry->d_inode->i_mutex);
                path_put(&nd.path);
-               return d;
+               goto out;
        }
        *path = nd.path;
+out:
+       putname(filename);
        return d;
 }
 
@@ -2297,19 +2344,13 @@ out:
 static int
 path_mountpoint(int dfd, const char *name, struct path *path, unsigned int flags)
 {
-       struct file *base = NULL;
        struct nameidata nd;
        int err;
 
-       err = path_init(dfd, name, flags | LOOKUP_PARENT, &nd, &base);
+       err = path_init(dfd, name, flags, &nd);
        if (unlikely(err))
                goto out;
 
-       current->total_link_count = 0;
-       err = link_path_walk(name, &nd);
-       if (err)
-               goto out;
-
        err = mountpoint_last(&nd, path);
        while (err > 0) {
                void *cookie;
@@ -2325,12 +2366,7 @@ path_mountpoint(int dfd, const char *name, struct path *path, unsigned int flags
                put_link(&nd, &link, cookie);
        }
 out:
-       if (base)
-               fput(base);
-
-       if (nd.root.mnt && !(nd.flags & LOOKUP_ROOT))
-               path_put(&nd.root);
-
+       path_cleanup(&nd);
        return err;
 }
 
@@ -2338,13 +2374,17 @@ static int
 filename_mountpoint(int dfd, struct filename *s, struct path *path,
                        unsigned int flags)
 {
-       int error = path_mountpoint(dfd, s->name, path, flags | LOOKUP_RCU);
+       int error;
+       if (IS_ERR(s))
+               return PTR_ERR(s);
+       error = path_mountpoint(dfd, s->name, path, flags | LOOKUP_RCU);
        if (unlikely(error == -ECHILD))
                error = path_mountpoint(dfd, s->name, path, flags);
        if (unlikely(error == -ESTALE))
                error = path_mountpoint(dfd, s->name, path, flags | LOOKUP_REVAL);
        if (likely(!error))
                audit_inode(s, path->dentry, 0);
+       putname(s);
        return error;
 }
 
@@ -2366,21 +2406,14 @@ int
 user_path_mountpoint_at(int dfd, const char __user *name, unsigned int flags,
                        struct path *path)
 {
-       struct filename *s = getname(name);
-       int error;
-       if (IS_ERR(s))
-               return PTR_ERR(s);
-       error = filename_mountpoint(dfd, s, path, flags);
-       putname(s);
-       return error;
+       return filename_mountpoint(dfd, getname(name), path, flags);
 }
 
 int
 kern_path_mountpoint(int dfd, const char *name, struct path *path,
                        unsigned int flags)
 {
-       struct filename s = {.name = name};
-       return filename_mountpoint(dfd, &s, path, flags);
+       return filename_mountpoint(dfd, getname_kernel(name), path, flags);
 }
 EXPORT_SYMBOL(kern_path_mountpoint);
 
@@ -3181,7 +3214,6 @@ out:
 static struct file *path_openat(int dfd, struct filename *pathname,
                struct nameidata *nd, const struct open_flags *op, int flags)
 {
-       struct file *base = NULL;
        struct file *file;
        struct path path;
        int opened = 0;
@@ -3198,12 +3230,7 @@ static struct file *path_openat(int dfd, struct filename *pathname,
                goto out;
        }
 
-       error = path_init(dfd, pathname->name, flags | LOOKUP_PARENT, nd, &base);
-       if (unlikely(error))
-               goto out;
-
-       current->total_link_count = 0;
-       error = link_path_walk(pathname->name, nd);
+       error = path_init(dfd, pathname->name, flags, nd);
        if (unlikely(error))
                goto out;
 
@@ -3229,10 +3256,7 @@ static struct file *path_openat(int dfd, struct filename *pathname,
                put_link(nd, &link, cookie);
        }
 out:
-       if (nd->root.mnt && !(nd->flags & LOOKUP_ROOT))
-               path_put(&nd->root);
-       if (base)
-               fput(base);
+       path_cleanup(nd);
        if (!(opened & FILE_OPENED)) {
                BUG_ON(!error);
                put_filp(file);
@@ -3269,7 +3293,7 @@ struct file *do_file_open_root(struct dentry *dentry, struct vfsmount *mnt,
 {
        struct nameidata nd;
        struct file *file;
-       struct filename filename = { .name = name };
+       struct filename *filename;
        int flags = op->lookup_flags | LOOKUP_ROOT;
 
        nd.root.mnt = mnt;
@@ -3278,15 +3302,20 @@ struct file *do_file_open_root(struct dentry *dentry, struct vfsmount *mnt,
        if (d_is_symlink(dentry) && op->intent & LOOKUP_OPEN)
                return ERR_PTR(-ELOOP);
 
-       file = path_openat(-1, &filename, &nd, op, flags | LOOKUP_RCU);
+       filename = getname_kernel(name);
+       if (unlikely(IS_ERR(filename)))
+               return ERR_CAST(filename);
+
+       file = path_openat(-1, filename, &nd, op, flags | LOOKUP_RCU);
        if (unlikely(file == ERR_PTR(-ECHILD)))
-               file = path_openat(-1, &filename, &nd, op, flags);
+               file = path_openat(-1, filename, &nd, op, flags);
        if (unlikely(file == ERR_PTR(-ESTALE)))
-               file = path_openat(-1, &filename, &nd, op, flags | LOOKUP_REVAL);
+               file = path_openat(-1, filename, &nd, op, flags | LOOKUP_REVAL);
+       putname(filename);
        return file;
 }
 
-struct dentry *kern_path_create(int dfd, const char *pathname,
+static struct dentry *filename_create(int dfd, struct filename *name,
                                struct path *path, unsigned int lookup_flags)
 {
        struct dentry *dentry = ERR_PTR(-EEXIST);
@@ -3301,7 +3330,7 @@ struct dentry *kern_path_create(int dfd, const char *pathname,
         */
        lookup_flags &= LOOKUP_REVAL;
 
-       error = do_path_lookup(dfd, pathname, LOOKUP_PARENT|lookup_flags, &nd);
+       error = filename_lookup(dfd, name, LOOKUP_PARENT|lookup_flags, &nd);
        if (error)
                return ERR_PTR(error);
 
@@ -3355,6 +3384,19 @@ out:
        path_put(&nd.path);
        return dentry;
 }
+
+struct dentry *kern_path_create(int dfd, const char *pathname,
+                               struct path *path, unsigned int lookup_flags)
+{
+       struct filename *filename = getname_kernel(pathname);
+       struct dentry *res;
+
+       if (IS_ERR(filename))
+               return ERR_CAST(filename);
+       res = filename_create(dfd, filename, path, lookup_flags);
+       putname(filename);
+       return res;
+}
 EXPORT_SYMBOL(kern_path_create);
 
 void done_path_create(struct path *path, struct dentry *dentry)
@@ -3373,7 +3415,7 @@ struct dentry *user_path_create(int dfd, const char __user *pathname,
        struct dentry *res;
        if (IS_ERR(tmp))
                return ERR_CAST(tmp);
-       res = kern_path_create(dfd, tmp->name, path, lookup_flags);
+       res = filename_create(dfd, tmp, path, lookup_flags);
        putname(tmp);
        return res;
 }