[PATCH] mtd: spi_nor: Fixes out of bound shift

Louis Rannou lrannou at baylibre.com
Tue Jan 31 07:04:50 PST 2023


Hello,

On 31/01/2023 09:17, Michael Walle wrote:
> Hi,
> 
> Am 2023-01-26 15:26, schrieb lrannou at baylibre.com:
>> From: Louis Rannou <lrannou at baylibre.com>
>>
>> spi_nor_set_erase_type is called twice in sfdp.c with a null size. The
>> return from ffs is 0 as well and the shift size becomes (2^32 - 1) 
>> which is
>> out of bound when applied to the << operator.
>>
>> This considers as illegal a call to this function with null size. It
>> creates a replacement spi_nor_mask_erase_type for explicit calls to mask
>> the erase type.
>>
>> Signed-off-by: Louis Rannou <lrannou at baylibre.com>
> 
> There is also this thread with a pending patch:
> https://lore.kernel.org/r/20211106075616.95401-1-tudor.ambarus@microchip.com/

Indeed, I did not see that.

My patch also misses the core.h update. Let's see if we get news from 
this pending patch. And then perhaps we'll see if I push a v2.

Regards,
Louis



More information about the linux-mtd mailing list