]> git.ipfire.org Git - thirdparty/linux.git/commit
fs/ntfs3: bound DeleteIndexEntryAllocation memmove length
authorKonstantin Komarov <almaz.alexandrovich@paragon-software.com>
Tue, 2 Jun 2026 13:21:03 +0000 (15:21 +0200)
committerKonstantin Komarov <almaz.alexandrovich@paragon-software.com>
Wed, 3 Jun 2026 14:37:02 +0000 (16:37 +0200)
commitfc4626bb3656362de8b0ecd56605d47a19ec3518
treeefff42ce099da90381cb6164eea431c9fca562ea
parentd1570c48f49a693974d000251030370ee2e83539
fs/ntfs3: bound DeleteIndexEntryAllocation memmove length

In do_action()'s DeleteIndexEntryAllocation case, e->size comes
from an on-disk INDEX_BUFFER entry.  When e->size makes
e + e->size point past hdr + hdr->used,
PtrOffset(e1, Add2Ptr(hdr, used)) returns a negative ptrdiff_t
that is silently cast to a quasi-infinite size_t when passed
to memmove().  The memmove then walks past the destination
buffer.

The sibling DeleteIndexEntryRoot case at fslog.c:3540-3543
already carries the corresponding guard:

if (PtrOffset(e1, Add2Ptr(hdr, used)) < esize ||
    Add2Ptr(e, esize) > Add2Ptr(lrh, rec_len) ||
    used + esize > le32_to_cpu(hdr->total)) {
goto dirty_vol;
}

Apply the same shape to the allocation-path case.  Also reject
esize == 0: memmove(e, e, ...) is a no-op and leaves
hdr->used unchanged, hiding a malformed entry from the
existing check_index_header() walk.

Reproduced under UML+KASAN on mainline 8d90b09e6741 by
mounting a crafted NTFS image: the unguarded memmove takes a
length of 0xffffffffffffff00 and the kernel oopses in
memmove+0x81/0x1a0 on the do_action+0x36a2 frame.

Fixes: b46acd6a6a62 ("fs/ntfs3: Add NTFS journal")
Cc: stable@vger.kernel.org
Assisted-by: Claude:claude-opus-4-7
Signed-off-by: Michael Bommarito <michael.bommarito@gmail.com>
[almaz.alexandrovich@paragon-software.com: clang-formatted the changes]
Signed-off-by: Konstantin Komarov <almaz.alexandrovich@paragon-software.com>
fs/ntfs3/fslog.c