Combine i_nb.blocks_hi with i_u.blocks_lo when computing
inode->i_blocks for compressed inodes, mirroring the startblk_hi
handling for unencoded inodes a few lines above.  Also evaluate
the shift in u64 to avoid truncation.

Fixes: 2e1473d5195f ("erofs: implement 48-bit block addressing for unencoded 
inodes")
Signed-off-by: Zhan Xusheng <[email protected]>
---
 fs/erofs/inode.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/fs/erofs/inode.c b/fs/erofs/inode.c
index a188c570087a..cf2f00e13cae 100644
--- a/fs/erofs/inode.c
+++ b/fs/erofs/inode.c
@@ -191,8 +191,9 @@ static int erofs_read_inode(struct inode *inode)
                err = -EFSCORRUPTED;
                goto err_out;
        } else {
-               inode->i_blocks = le32_to_cpu(copied.i_u.blocks_lo) <<
-                               (sb->s_blocksize_bits - 9);
+               inode->i_blocks = ((u64)le16_to_cpu(copied.i_nb.blocks_hi) << 
32 |
+                                  le32_to_cpu(copied.i_u.blocks_lo)) <<
+                                 (sb->s_blocksize_bits - 9);
        }
 
        if (vi->datalayout == EROFS_INODE_CHUNK_BASED) {
-- 
2.43.0


Reply via email to