Skip to content

Commit ee1b421

Browse files
chaseyugregkh
authored andcommitted
f2fs: fix to do sanity check on sit_bitmap_size
commit 5db0d25 upstream. w/ below testcase, resize will generate a corrupted image which contains inconsistent metadata, so when mounting such image, it will trigger kernel panic: touch img truncate -s $((512*1024*1024*1024)) img mkfs.f2fs -f img $((256*1024*1024)) resize.f2fs -s -i img -t $((1024*1024*1024)) mount img /mnt/f2fs ------------[ cut here ]------------ kernel BUG at fs/f2fs/segment.h:863! Oops: invalid opcode: 0000 [#1] SMP PTI CPU: 11 UID: 0 PID: 3922 Comm: mount Not tainted 6.15.0-rc1+ #191 PREEMPT(voluntary) Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS 1.16.3-debian-1.16.3-2 04/01/2014 RIP: 0010:f2fs_ra_meta_pages+0x47c/0x490 Call Trace: f2fs_build_segment_manager+0x11c3/0x2600 f2fs_fill_super+0xe97/0x2840 mount_bdev+0xf4/0x140 legacy_get_tree+0x2b/0x50 vfs_get_tree+0x29/0xd0 path_mount+0x487/0xaf0 __x64_sys_mount+0x116/0x150 do_syscall_64+0x82/0x190 entry_SYSCALL_64_after_hwframe+0x76/0x7e RIP: 0033:0x7fdbfde1bcfe The reaseon is: sit_i->bitmap_size is 192, so size of sit bitmap is 192*8=1536, at maximum there are 1536 sit blocks, however MAIN_SEGS is 261893, so that sit_blk_cnt is 4762, build_sit_entries() -> current_sit_addr() tries to access out-of-boundary in sit_bitmap at offset from [1536, 4762), once sit_bitmap and sit_bitmap_mirror is not the same, it will trigger f2fs_bug_on(). Let's add sanity check in f2fs_sanity_check_ckpt() to avoid panic. Cc: [email protected] Signed-off-by: Chao Yu <[email protected]> Signed-off-by: Jaegeuk Kim <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
1 parent aaa644e commit ee1b421

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

fs/f2fs/super.c

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3659,6 +3659,7 @@ int f2fs_sanity_check_ckpt(struct f2fs_sb_info *sbi)
36593659
block_t user_block_count, valid_user_blocks;
36603660
block_t avail_node_count, valid_node_count;
36613661
unsigned int nat_blocks, nat_bits_bytes, nat_bits_blocks;
3662+
unsigned int sit_blk_cnt;
36623663
int i, j;
36633664

36643665
total = le32_to_cpu(raw_super->segment_count);
@@ -3770,6 +3771,13 @@ int f2fs_sanity_check_ckpt(struct f2fs_sb_info *sbi)
37703771
return 1;
37713772
}
37723773

3774+
sit_blk_cnt = DIV_ROUND_UP(main_segs, SIT_ENTRY_PER_BLOCK);
3775+
if (sit_bitmap_size * 8 < sit_blk_cnt) {
3776+
f2fs_err(sbi, "Wrong bitmap size: sit: %u, sit_blk_cnt:%u",
3777+
sit_bitmap_size, sit_blk_cnt);
3778+
return 1;
3779+
}
3780+
37733781
cp_pack_start_sum = __start_sum_addr(sbi);
37743782
cp_payload = __cp_payload(sbi);
37753783
if (cp_pack_start_sum < cp_payload + 1 ||

0 commit comments

Comments
 (0)