f2fs: compress: fix to check validity of i_compress_flag field
authorChao Yu <chao@kernel.org>
Wed, 17 May 2023 03:41:39 +0000 (11:41 +0800)
committerJaegeuk Kim <jaegeuk@kernel.org>
Mon, 26 Jun 2023 13:07:09 +0000 (06:07 -0700)
The last valid compress related field is i_compress_flag, check its
validity instead of i_log_cluster_size.

Signed-off-by: Chao Yu <chao@kernel.org>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
fs/f2fs/inode.c

index bb6c3733d1041d6b5c9028cfdf87e41e5870be0f..6cacc8c55dec6df920d75d93d6076b84616ed115 100644 (file)
@@ -362,7 +362,7 @@ static bool sanity_check_inode(struct inode *inode, struct page *node_page)
        if (f2fs_has_extra_attr(inode) && f2fs_sb_has_compression(sbi) &&
                        fi->i_flags & F2FS_COMPR_FL &&
                        F2FS_FITS_IN_INODE(ri, fi->i_extra_isize,
-                                               i_log_cluster_size))
+                                               i_compress_flag))
                return sanity_check_compress_inode(inode, ri);
 
        return true;
@@ -491,7 +491,7 @@ static int do_read_inode(struct inode *inode)
        if (f2fs_has_extra_attr(inode) && f2fs_sb_has_compression(sbi) &&
                                        (fi->i_flags & F2FS_COMPR_FL)) {
                if (F2FS_FITS_IN_INODE(ri, fi->i_extra_isize,
-                                       i_log_cluster_size)) {
+                                       i_compress_flag)) {
                        unsigned short compress_flag;
 
                        atomic_set(&fi->i_compr_blocks,
@@ -729,7 +729,7 @@ void f2fs_update_inode(struct inode *inode, struct page *node_page)
 
                if (f2fs_sb_has_compression(F2FS_I_SB(inode)) &&
                        F2FS_FITS_IN_INODE(ri, F2FS_I(inode)->i_extra_isize,
-                                                       i_log_cluster_size)) {
+                                                       i_compress_flag)) {
                        unsigned short compress_flag;
 
                        ri->i_compr_blocks =