[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <c8190aae-25d8-469d-8efa-6d1ff9482347@embeddedor.com>
Date: Sat, 20 Dec 2025 16:12:33 +0900
From: "Gustavo A. R. Silva" <gustavo@...eddedor.com>
To: Raag Jadav <raag.jadav@...el.com>,
"Gustavo A. R. Silva" <gustavoars@...nel.org>
Cc: Randy Dunlap <rdunlap@...radead.org>,
Alexander Usyskin <alexander.usyskin@...el.com>,
Miquel Raynal <miquel.raynal@...tlin.com>,
Richard Weinberger <richard@....at>, Vignesh Raghavendra <vigneshr@...com>,
Rodrigo Vivi <rodrigo.vivi@...el.com>, Tomas Winkler <tomasw@...il.com>,
linux-mtd@...ts.infradead.org, linux-kernel@...r.kernel.org,
linux-hardening@...r.kernel.org
Subject: Re: [PATCH] mtd: intel-dg: fix array-index-out-of-bounds in
intel_dg_mtd_probe()
On 12/20/25 16:07, Raag Jadav wrote:
> On Sat, Dec 20, 2025 at 03:41:49PM +0900, Gustavo A. R. Silva wrote:
>> Fix the UBSAN: array-index-out-of-bounds issue below by updating
>> counter nvm->nregions before the first access to flexible-array
>> member nvm->regions[].
>>
>> from kernel bugzilla:
>> https://bugzilla.kernel.org/show_bug.cgi?id=220823
>>
>> Dec 15 22:01:52 orpheus kernel: UBSAN: array-index-out-of-bounds in /var/tmp/portage/sys-kernel/gentoo-kernel-6.18.1/work/linux-6.18/drivers/mtd/devices/mtd_intel_dg.c:750:15
>>
>> Notice that this flexible array is annotated with the counted_by()
>> attribute, hence the counter must always be updated before the
>> first access to the array.
>
> Already fixed[1], but not sure if it's landed yet.
>
> [1] https://lore.kernel.org/linux-mtd/20251111-mtd-nregions-v1-1-61db61e78c63@intel.com/
Great! :)
Thanks
-Gustavo
Powered by blists - more mailing lists