xfs: rename code to error in xfs_ioctl_setattr
authorDarrick J. Wong <djwong@kernel.org>
Sat, 30 Jan 2021 01:14:33 +0000 (17:14 -0800)
committerDarrick J. Wong <djwong@kernel.org>
Wed, 3 Feb 2021 17:18:49 +0000 (09:18 -0800)
Rename the 'code' variable to 'error' to follow the naming convention of
most other functions in xfs.

Signed-off-by: Darrick J. Wong <djwong@kernel.org>
Reviewed-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Brian Foster <bfoster@redhat.com>
fs/xfs/xfs_ioctl.c

index 78ee201eb7cb9f713a74be658b3bfc888513e3e2..38ee66d999d8cdc9c51fe63c4e1f9dfa4435b7f5 100644 (file)
@@ -1435,13 +1435,13 @@ xfs_ioctl_setattr(
        struct xfs_trans        *tp;
        struct xfs_dquot        *pdqp = NULL;
        struct xfs_dquot        *olddquot = NULL;
-       int                     code;
+       int                     error;
 
        trace_xfs_ioctl_setattr(ip);
 
-       code = xfs_ioctl_setattr_check_projid(ip, fa);
-       if (code)
-               return code;
+       error = xfs_ioctl_setattr_check_projid(ip, fa);
+       if (error)
+               return error;
 
        /*
         * If disk quotas is on, we make sure that the dquots do exist on disk,
@@ -1452,36 +1452,36 @@ xfs_ioctl_setattr(
         * because the i_*dquot fields will get updated anyway.
         */
        if (XFS_IS_QUOTA_ON(mp)) {
-               code = xfs_qm_vop_dqalloc(ip, VFS_I(ip)->i_uid,
+               error = xfs_qm_vop_dqalloc(ip, VFS_I(ip)->i_uid,
                                VFS_I(ip)->i_gid, fa->fsx_projid,
                                XFS_QMOPT_PQUOTA, NULL, NULL, &pdqp);
-               if (code)
-                       return code;
+               if (error)
+                       return error;
        }
 
        xfs_ioctl_setattr_prepare_dax(ip, fa);
 
        tp = xfs_ioctl_setattr_get_trans(ip, pdqp);
        if (IS_ERR(tp)) {
-               code = PTR_ERR(tp);
+               error = PTR_ERR(tp);
                goto error_free_dquots;
        }
 
        xfs_fill_fsxattr(ip, false, &old_fa);
-       code = vfs_ioc_fssetxattr_check(VFS_I(ip), &old_fa, fa);
-       if (code)
+       error = vfs_ioc_fssetxattr_check(VFS_I(ip), &old_fa, fa);
+       if (error)
                goto error_trans_cancel;
 
-       code = xfs_ioctl_setattr_check_extsize(ip, fa);
-       if (code)
+       error = xfs_ioctl_setattr_check_extsize(ip, fa);
+       if (error)
                goto error_trans_cancel;
 
-       code = xfs_ioctl_setattr_check_cowextsize(ip, fa);
-       if (code)
+       error = xfs_ioctl_setattr_check_cowextsize(ip, fa);
+       if (error)
                goto error_trans_cancel;
 
-       code = xfs_ioctl_setattr_xflags(tp, ip, fa);
-       if (code)
+       error = xfs_ioctl_setattr_xflags(tp, ip, fa);
+       if (error)
                goto error_trans_cancel;
 
        /*
@@ -1521,7 +1521,7 @@ xfs_ioctl_setattr(
        else
                ip->i_d.di_cowextsize = 0;
 
-       code = xfs_trans_commit(tp);
+       error = xfs_trans_commit(tp);
 
        /*
         * Release any dquot(s) the inode had kept before chown.
@@ -1529,13 +1529,13 @@ xfs_ioctl_setattr(
        xfs_qm_dqrele(olddquot);
        xfs_qm_dqrele(pdqp);
 
-       return code;
+       return error;
 
 error_trans_cancel:
        xfs_trans_cancel(tp);
 error_free_dquots:
        xfs_qm_dqrele(pdqp);
-       return code;
+       return error;
 }
 
 STATIC int