mqueue: switch to vfs_mkobj(), quit abusing ->d_fsdata
authorAl Viro <viro@zeniv.linux.org.uk>
Fri, 1 Dec 2017 22:26:05 +0000 (17:26 -0500)
committerAl Viro <viro@zeniv.linux.org.uk>
Fri, 5 Jan 2018 16:54:33 +0000 (11:54 -0500)
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
ipc/mqueue.c

index 9649ecd8a73a704fe07e3d678ac96723d33aec05..e011ccd4c798d6028e5880fe7a07b7c89e56b4fd 100644 (file)
@@ -416,11 +416,11 @@ static void mqueue_evict_inode(struct inode *inode)
                put_ipc_ns(ipc_ns);
 }
 
-static int mqueue_create(struct inode *dir, struct dentry *dentry,
-                               umode_t mode, bool excl)
+static int mqueue_create_attr(struct dentry *dentry, umode_t mode, void *arg)
 {
+       struct inode *dir = dentry->d_parent->d_inode;
        struct inode *inode;
-       struct mq_attr *attr = dentry->d_fsdata;
+       struct mq_attr *attr = arg;
        int error;
        struct ipc_namespace *ipc_ns;
 
@@ -461,6 +461,12 @@ out_unlock:
        return error;
 }
 
+static int mqueue_create(struct inode *dir, struct dentry *dentry,
+                               umode_t mode, bool excl)
+{
+       return mqueue_create_attr(dentry, mode, NULL);
+}
+
 static int mqueue_unlink(struct inode *dir, struct dentry *dentry)
 {
        struct inode *inode = d_inode(dentry);
@@ -732,8 +738,6 @@ static struct file *do_create(struct ipc_namespace *ipc_ns, struct inode *dir,
                ret = mq_attr_ok(ipc_ns, attr);
                if (ret)
                        return ERR_PTR(ret);
-               /* store for use during create */
-               path->dentry->d_fsdata = attr;
        } else {
                struct mq_attr def_attr;
 
@@ -746,9 +750,8 @@ static struct file *do_create(struct ipc_namespace *ipc_ns, struct inode *dir,
                        return ERR_PTR(ret);
        }
 
-       mode &= ~current_umask();
-       ret = vfs_create(dir, path->dentry, mode, true);
-       path->dentry->d_fsdata = NULL;
+       ret = vfs_mkobj(path->dentry, mode & ~current_umask(),
+                       mqueue_create_attr, attr);
        if (ret)
                return ERR_PTR(ret);
        return dentry_open(path, oflag, cred);