* a negative ntfs superblock mft_lcn can cause a wild memory reference
@ 2026-08-17 21:03 rtm
2026-08-18 9:48 ` Hyunchul Lee
0 siblings, 1 reply; 2+ messages in thread
From: rtm @ 2026-08-17 21:03 UTC (permalink / raw)
To: Namjae Jeon, Hyunchul Lee; +Cc: linux-fsdevel
If the ntfs superblock's mft_lcn is negative, that will pass tests
such as
ll = le64_to_cpu(b->mft_lcn);
if (ll >= vol->nr_clusters) {
and
vol->mft_zone_end = vol->mft_lcn + mft_zone_size;
while (vol->mft_zone_end >= vol->nr_clusters) {
because the variables are signed.
The attached corrupt NTFS image exploits this to cause index to be
out of bounds in ntfs_cluster_alloc(), e.g. for
if (vol->lcn_empty_bits_per_page[index] == 0)
goto next_bmp_pos;
index is derived from mft_lcn via mft_zone_end, data1_zone_pos,
zone_start, and bmp_pos.
# wget http://www.rtmrtm.org/rtm/ntfs8b.img.gz
# gunzip ntfs8b.img.gz
# cp ntfs8b.img junk
# mount -t ntfs -o loop junk /mnt
# echo x > /mnt/x
ntfs: (device loop0): check_mft_mirror(): $MFT and $MFTMirr record 0 do not match. Run chkdsk.
BUG: unable to handle page fault for address: ffff80010e38ef58
#PF: supervisor read access in kernel mode
#PF: error_code(0x0000) - not-present page
PGD 0 P4D 0
Oops: Oops: 0000 [#1] SMP DEBUG_PAGEALLOC PTI
CPU: 10 UID: 0 PID: 1264 Comm: tcsh Not tainted 7.2.0-rc7-00016-g3d6d817622b0 #40 PREEMPT(full)
Hardware name: FreeBSD BHYVE/BHYVE, BIOS 14.0 10/17/2021
RIP: 0010:ntfs_cluster_alloc+0x376/0x12a0
Call Trace:
<TASK>
? inode_io_list_move_locked+0xa1/0xe0
? folio_mark_dirty+0x33/0x60
ntfs_mft_data_extend_allocation_nolock+0x15b/0xa30
ntfs_mft_record_alloc+0x921/0x1790
__ntfs_create+0x1c4/0xd40
? ntfs_attr_put_search_ctx+0x3b/0x50
? kmem_cache_free+0x207/0x2f0
? __mark_inode_dirty+0x16a/0x3b0
ntfs_create+0xc1/0x170
path_openat+0xfe0/0x12f0
? do_syscall_64+0xf9/0x540
? do_getname+0x2e/0x180
do_file_open+0xdf/0x1a0
do_sys_openat2+0x79/0xe0
__x64_sys_creat+0x49/0x70
x64_sys_call+0x1bfa/0x2130
do_syscall_64+0xf9/0x540
? exc_page_fault+0x76/0x130
entry_SYSCALL_64_after_hwframe+0x77/0x7f
Robert Morris
rtm@mit.edu
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: a negative ntfs superblock mft_lcn can cause a wild memory reference
2026-08-17 21:03 a negative ntfs superblock mft_lcn can cause a wild memory reference rtm
@ 2026-08-18 9:48 ` Hyunchul Lee
0 siblings, 0 replies; 2+ messages in thread
From: Hyunchul Lee @ 2026-08-18 9:48 UTC (permalink / raw)
To: rtm; +Cc: Namjae Jeon, linux-fsdevel, Hyunchul Lee
Hi Robert,
I have submitted a patch to fix this issue:
https://lore.kernel.org/all/20260818-fix-negative-mft-lcn-v1-1-b7b0b206f2da@gmail.com/
2026년 8월 18일 (화) 오전 6:03, <rtm@csail.mit.edu>님이 작성:
>
> If the ntfs superblock's mft_lcn is negative, that will pass tests
> such as
>
> ll = le64_to_cpu(b->mft_lcn);
> if (ll >= vol->nr_clusters) {
>
> and
>
> vol->mft_zone_end = vol->mft_lcn + mft_zone_size;
> while (vol->mft_zone_end >= vol->nr_clusters) {
>
> because the variables are signed.
>
> The attached corrupt NTFS image exploits this to cause index to be
> out of bounds in ntfs_cluster_alloc(), e.g. for
>
> if (vol->lcn_empty_bits_per_page[index] == 0)
> goto next_bmp_pos;
>
> index is derived from mft_lcn via mft_zone_end, data1_zone_pos,
> zone_start, and bmp_pos.
>
> # wget http://www.rtmrtm.org/rtm/ntfs8b.img.gz
> # gunzip ntfs8b.img.gz
> # cp ntfs8b.img junk
> # mount -t ntfs -o loop junk /mnt
> # echo x > /mnt/x
> ntfs: (device loop0): check_mft_mirror(): $MFT and $MFTMirr record 0 do not match. Run chkdsk.
> BUG: unable to handle page fault for address: ffff80010e38ef58
> #PF: supervisor read access in kernel mode
> #PF: error_code(0x0000) - not-present page
> PGD 0 P4D 0
> Oops: Oops: 0000 [#1] SMP DEBUG_PAGEALLOC PTI
> CPU: 10 UID: 0 PID: 1264 Comm: tcsh Not tainted 7.2.0-rc7-00016-g3d6d817622b0 #40 PREEMPT(full)
> Hardware name: FreeBSD BHYVE/BHYVE, BIOS 14.0 10/17/2021
> RIP: 0010:ntfs_cluster_alloc+0x376/0x12a0
> Call Trace:
> <TASK>
> ? inode_io_list_move_locked+0xa1/0xe0
> ? folio_mark_dirty+0x33/0x60
> ntfs_mft_data_extend_allocation_nolock+0x15b/0xa30
> ntfs_mft_record_alloc+0x921/0x1790
> __ntfs_create+0x1c4/0xd40
> ? ntfs_attr_put_search_ctx+0x3b/0x50
> ? kmem_cache_free+0x207/0x2f0
> ? __mark_inode_dirty+0x16a/0x3b0
> ntfs_create+0xc1/0x170
> path_openat+0xfe0/0x12f0
> ? do_syscall_64+0xf9/0x540
> ? do_getname+0x2e/0x180
> do_file_open+0xdf/0x1a0
> do_sys_openat2+0x79/0xe0
> __x64_sys_creat+0x49/0x70
> x64_sys_call+0x1bfa/0x2130
> do_syscall_64+0xf9/0x540
> ? exc_page_fault+0x76/0x130
> entry_SYSCALL_64_after_hwframe+0x77/0x7f
>
> Robert Morris
> rtm@mit.edu
>
--
Thanks,
Hyunchul
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-08-18 9:48 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-08-17 21:03 a negative ntfs superblock mft_lcn can cause a wild memory reference rtm
2026-08-18 9:48 ` Hyunchul Lee
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox