ext4: initialize quota before expanding inode in setproject ioctl
authorJan Kara <jack@suse.cz>
Wed, 7 Dec 2022 11:59:27 +0000 (12:59 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 12 Jan 2023 10:59:03 +0000 (11:59 +0100)
commit 1485f726c6dec1a1f85438f2962feaa3d585526f upstream.

Make sure we initialize quotas before possibly expanding inode space
(and thus maybe needing to allocate external xattr block) in
ext4_ioctl_setproject(). This prevents not accounting the necessary
block allocation.

Signed-off-by: Jan Kara <jack@suse.cz>
Cc: stable@kernel.org
Link: https://lore.kernel.org/r/20221207115937.26601-1-jack@suse.cz
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
fs/ext4/ioctl.c

index b6c480e..2e6d03e 100644 (file)
@@ -492,6 +492,10 @@ static int ext4_ioctl_setproject(struct inode *inode, __u32 projid)
        if (ext4_is_quota_file(inode))
                return err;
 
+       err = dquot_initialize(inode);
+       if (err)
+               return err;
+
        err = ext4_get_inode_loc(inode, &iloc);
        if (err)
                return err;
@@ -507,10 +511,6 @@ static int ext4_ioctl_setproject(struct inode *inode, __u32 projid)
                brelse(iloc.bh);
        }
 
-       err = dquot_initialize(inode);
-       if (err)
-               return err;
-
        handle = ext4_journal_start(inode, EXT4_HT_QUOTA,
                EXT4_QUOTA_INIT_BLOCKS(sb) +
                EXT4_QUOTA_DEL_BLOCKS(sb) + 3);