[XFS] cleanup vnode use in xfs_symlink and xfs_rename
[pandora-kernel.git] / fs / xfs / linux-2.6 / xfs_iops.c
index cc4abd3..215158c 100644 (file)
@@ -241,18 +241,6 @@ xfs_init_security(
        return error;
 }
 
-/*
- * Determine whether a process has a valid fs_struct (kernel daemons
- * like knfsd don't have an fs_struct).
- *
- * XXX(hch):  nfsd is broken, better fix it instead.
- */
-STATIC_INLINE int
-xfs_has_fs_struct(struct task_struct *task)
-{
-       return (task->fs != init_task.fs);
-}
-
 STATIC void
 xfs_cleanup_inode(
        struct inode    *dir,
@@ -284,8 +272,8 @@ xfs_vn_mknod(
        int             mode,
        dev_t           rdev)
 {
-       struct inode    *ip;
-       bhv_vnode_t     *vp = NULL, *dvp = vn_from_inode(dir);
+       struct inode    *inode;
+       struct xfs_inode *ip = NULL;
        xfs_acl_t       *default_acl = NULL;
        attrexists_t    test_default_acl = _ACL_DEFAULT_EXISTS;
        int             error;
@@ -297,59 +285,65 @@ xfs_vn_mknod(
        if (unlikely(!sysv_valid_dev(rdev) || MAJOR(rdev) & ~0x1ff))
                return -EINVAL;
 
-       if (unlikely(test_default_acl && test_default_acl(dvp))) {
+       if (test_default_acl && test_default_acl(dir)) {
                if (!_ACL_ALLOC(default_acl)) {
                        return -ENOMEM;
                }
-               if (!_ACL_GET_DEFAULT(dvp, default_acl)) {
+               if (!_ACL_GET_DEFAULT(dir, default_acl)) {
                        _ACL_FREE(default_acl);
                        default_acl = NULL;
                }
        }
 
-       if (IS_POSIXACL(dir) && !default_acl && xfs_has_fs_struct(current))
+       if (IS_POSIXACL(dir) && !default_acl)
                mode &= ~current->fs->umask;
 
        switch (mode & S_IFMT) {
-       case S_IFCHR: case S_IFBLK: case S_IFIFO: case S_IFSOCK:
+       case S_IFCHR:
+       case S_IFBLK:
+       case S_IFIFO:
+       case S_IFSOCK:
                rdev = sysv_encode_dev(rdev);
        case S_IFREG:
-               error = xfs_create(XFS_I(dir), dentry, mode, rdev, &vp, NULL);
+               error = xfs_create(XFS_I(dir), dentry, mode, rdev, &ip, NULL);
                break;
        case S_IFDIR:
-               error = xfs_mkdir(XFS_I(dir), dentry, mode, &vp, NULL);
+               error = xfs_mkdir(XFS_I(dir), dentry, mode, &ip, NULL);
                break;
        default:
                error = EINVAL;
                break;
        }
 
-       if (unlikely(!error)) {
-               error = xfs_init_security(vp, dir);
-               if (error)
-                       xfs_cleanup_inode(dir, vp, dentry, mode);
-       }
+       if (unlikely(error))
+               goto out_free_acl;
 
-       if (unlikely(default_acl)) {
-               if (!error) {
-                       error = _ACL_INHERIT(vp, mode, default_acl);
-                       if (!error)
-                               xfs_iflags_set(XFS_I(vp), XFS_IMODIFIED);
-                       else
-                               xfs_cleanup_inode(dir, vp, dentry, mode);
-               }
+       inode = ip->i_vnode;
+
+       error = xfs_init_security(inode, dir);
+       if (unlikely(error))
+               goto out_cleanup_inode;
+
+       if (default_acl) {
+               error = _ACL_INHERIT(inode, mode, default_acl);
+               if (unlikely(error))
+                       goto out_cleanup_inode;
+               xfs_iflags_set(ip, XFS_IMODIFIED);
                _ACL_FREE(default_acl);
        }
 
-       if (likely(!error)) {
-               ASSERT(vp);
-               ip = vn_to_inode(vp);
 
-               if (S_ISDIR(mode))
-                       xfs_validate_fields(ip);
-               d_instantiate(dentry, ip);
-               xfs_validate_fields(dir);
-       }
+       if (S_ISDIR(mode))
+               xfs_validate_fields(inode);
+       d_instantiate(dentry, inode);
+       xfs_validate_fields(dir);
+       return -error;
+
+ out_cleanup_inode:
+       xfs_cleanup_inode(dir, inode, dentry, mode);
+ out_free_acl:
+       if (default_acl)
+               _ACL_FREE(default_acl);
        return -error;
 }
 
@@ -401,23 +395,22 @@ xfs_vn_link(
        struct inode    *dir,
        struct dentry   *dentry)
 {
-       struct inode    *ip;    /* inode of guy being linked to */
-       bhv_vnode_t     *vp;    /* vp of name being linked */
+       struct inode    *inode; /* inode of guy being linked to */
        int             error;
 
-       ip = old_dentry->d_inode;       /* inode being linked to */
-       vp = vn_from_inode(ip);
+       inode = old_dentry->d_inode;
 
-       VN_HOLD(vp);
-       error = xfs_link(XFS_I(dir), vp, dentry);
+       igrab(inode);
+       error = xfs_link(XFS_I(dir), XFS_I(inode), dentry);
        if (unlikely(error)) {
-               VN_RELE(vp);
-       } else {
-               xfs_iflags_set(XFS_I(dir), XFS_IMODIFIED);
-               xfs_validate_fields(ip);
-               d_instantiate(dentry, ip);
+               iput(inode);
+               return -error;
        }
-       return -error;
+
+       xfs_iflags_set(XFS_I(dir), XFS_IMODIFIED);
+       xfs_validate_fields(inode);
+       d_instantiate(dentry, inode);
+       return 0;
 }
 
 STATIC int
@@ -444,29 +437,33 @@ xfs_vn_symlink(
        struct dentry   *dentry,
        const char      *symname)
 {
-       struct inode    *ip;
-       bhv_vnode_t     *cvp;   /* used to lookup symlink to put in dentry */
+       struct inode    *inode;
+       struct xfs_inode *cip = NULL;
        int             error;
        mode_t          mode;
 
-       cvp = NULL;
-
        mode = S_IFLNK |
                (irix_symlink_mode ? 0777 & ~current->fs->umask : S_IRWXUGO);
 
        error = xfs_symlink(XFS_I(dir), dentry, (char *)symname, mode,
-                           &cvp, NULL);
-       if (likely(!error && cvp)) {
-               error = xfs_init_security(cvp, dir);
-               if (likely(!error)) {
-                       ip = vn_to_inode(cvp);
-                       d_instantiate(dentry, ip);
-                       xfs_validate_fields(dir);
-                       xfs_validate_fields(ip);
-               } else {
-                       xfs_cleanup_inode(dir, cvp, dentry, 0);
-               }
-       }
+                           &cip, NULL);
+       if (unlikely(error))
+               goto out;
+
+       inode = cip->i_vnode;
+
+       error = xfs_init_security(inode, dir);
+       if (unlikely(error))
+               goto out_cleanup_inode;
+
+       d_instantiate(dentry, inode);
+       xfs_validate_fields(dir);
+       xfs_validate_fields(inode);
+       return 0;
+
+ out_cleanup_inode:
+       xfs_cleanup_inode(dir, inode, dentry, 0);
+ out:
        return -error;
 }
 
@@ -494,12 +491,9 @@ xfs_vn_rename(
        struct dentry   *ndentry)
 {
        struct inode    *new_inode = ndentry->d_inode;
-       bhv_vnode_t     *tvp;   /* target directory */
        int             error;
 
-       tvp = vn_from_inode(ndir);
-
-       error = xfs_rename(XFS_I(odir), odentry, tvp, ndentry);
+       error = xfs_rename(XFS_I(odir), odentry, XFS_I(ndir), ndentry);
        if (likely(!error)) {
                if (new_inode)
                        xfs_validate_fields(new_inode);