]> git.ipfire.org Git - thirdparty/kernel/linux.git/commit
fs/ntfs3: fix wrong LCN in run_remove_range() when splitting a run
authorZhan Xusheng <zhanxusheng1024@gmail.com>
Fri, 8 May 2026 09:52:45 +0000 (17:52 +0800)
committerKonstantin Komarov <almaz.alexandrovich@paragon-software.com>
Tue, 2 Jun 2026 15:02:28 +0000 (17:02 +0200)
commit36c7276816ed4266c155b71b1fa747b2785f23f7
tree5c95b6a50a2479d92663d69bcf3e415745569c75
parent57382ec6ac63b63dce2789e835fded28b698ae79
fs/ntfs3: fix wrong LCN in run_remove_range() when splitting a run

When run_remove_range() removes a middle portion of a non-sparse run,
it splits the run into head and tail parts.  The tail is inserted via
run_add_entry() but uses the original r->lcn as its starting LCN
instead of advancing it by the split offset.

For example, removing VCN range [10, 20) from a run
{vcn=0, lcn=100, len=30} should produce:
  {vcn=0,  lcn=100, len=10}   (head)
  {vcn=20, lcn=120, len=10}   (tail, lcn advanced by 20)

But the current code produces:
  {vcn=0,  lcn=100, len=10}
  {vcn=20, lcn=100, len=10}   (wrong: points to same physical clusters)

This creates overlapping physical mappings in the in-memory run tree,
which can corrupt cluster allocation decisions and lead to data
corruption.

The correct pattern is already used in run_insert_range():
  CLST lcn2 = r->lcn == SPARSE_LCN ? SPARSE_LCN : (r->lcn + len1);

Apply the same logic in run_remove_range().

Fixes: 10d7c95af043 ("fs/ntfs3: add delayed-allocation (delalloc) support")
Signed-off-by: Zhan Xusheng <zhanxusheng@xiaomi.com>
Signed-off-by: Konstantin Komarov <almaz.alexandrovich@paragon-software.com>
fs/ntfs3/run.c