linux-kernel - Re: [syzbot] Re: [syzbot] [ntfs3?] KASAN: slab-out-of-bounds Read in mi_enum

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CACT4Y+ZWjUE6-q1YF=ZaPjKgGZBw4JLD1v2DphSgCAVf1RzE8g@mail.gmail.com>
Date: Thu, 4 Jul 2024 15:44:02 +0200
From: Dmitry Vyukov <dvyukov@...gle.com>
To: syzbot <syzbot+a426cde6dee8c2884b0b@...kaller.appspotmail.com>
Cc: linux-kernel@...r.kernel.org, syzkaller-bugs@...glegroups.com
Subject: Re: [syzbot] Re: [syzbot] [ntfs3?] KASAN: slab-out-of-bounds Read in mi_enum_attr

On Fri, 2 Feb 2024 at 03:49, syzbot
<syzbot+a426cde6dee8c2884b0b@...kaller.appspotmail.com> wrote:
>
> For archival purposes, forwarding an incoming command email to
> linux-kernel@...r.kernel.org, syzkaller-bugs@...glegroups.com.
>
> ***
>
> Subject: Re: [syzbot] [ntfs3?] KASAN: slab-out-of-bounds Read in mi_enum_attr
> Author: lizhi.xu@...driver.com
>
> #syz test https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
>
> diff --git a/fs/ntfs3/record.c b/fs/ntfs3/record.c
> index 53629b1f65e9..a435df98c2b1 100644
> --- a/fs/ntfs3/record.c
> +++ b/fs/ntfs3/record.c
> @@ -243,14 +243,14 @@ struct ATTRIB *mi_enum_attr(struct mft_inode *mi, struct ATTRIB *attr)
>                 off += asize;
>         }
>
> -       asize = le32_to_cpu(attr->size);
> -
>         /* Can we use the first field (attr->type). */
>         if (off + 8 > used) {
>                 static_assert(ALIGN(sizeof(enum ATTR_TYPE), 8) == 8);
>                 return NULL;
>         }
>
> +       asize = le32_to_cpu(attr->size);
> +
>         if (attr->type == ATTR_END) {
>                 /* End of enumeration. */
>                 return NULL;

Hi Lizhi,

I don't see this fix mailed as a patch. Do you plan to submit it officially?