f2fs: handle dqget error in f2fs_transfer_project_quota()
authorYangtao Li <frank.li@vivo.com>
Tue, 21 Feb 2023 14:45:50 +0000 (22:45 +0800)
committerJaegeuk Kim <jaegeuk@kernel.org>
Wed, 29 Mar 2023 22:17:37 +0000 (15:17 -0700)
We should set the error code when dqget() failed.

Fixes: 2c1d03056991 ("f2fs: support F2FS_IOC_FS{GET,SET}XATTR")
Signed-off-by: Yangtao Li <frank.li@vivo.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
fs/f2fs/file.c

index 12d5502..3fbfdfd 100644 (file)
@@ -3009,15 +3009,16 @@ int f2fs_transfer_project_quota(struct inode *inode, kprojid_t kprojid)
        struct dquot *transfer_to[MAXQUOTAS] = {};
        struct f2fs_sb_info *sbi = F2FS_I_SB(inode);
        struct super_block *sb = sbi->sb;
-       int err = 0;
+       int err;
 
        transfer_to[PRJQUOTA] = dqget(sb, make_kqid_projid(kprojid));
-       if (!IS_ERR(transfer_to[PRJQUOTA])) {
-               err = __dquot_transfer(inode, transfer_to);
-               if (err)
-                       set_sbi_flag(sbi, SBI_QUOTA_NEED_REPAIR);
-               dqput(transfer_to[PRJQUOTA]);
-       }
+       if (IS_ERR(transfer_to[PRJQUOTA]))
+               return PTR_ERR(transfer_to[PRJQUOTA]);
+
+       err = __dquot_transfer(inode, transfer_to);
+       if (err)
+               set_sbi_flag(sbi, SBI_QUOTA_NEED_REPAIR);
+       dqput(transfer_to[PRJQUOTA]);
        return err;
 }