[lvc-project] [PATCH 3/3] drivers/gpu/drm/nouveau/nvkm/subdev/mxm/base.c: BUFFER_OVERFLOW

[Fedor Pchelkin]

Добрый день!

[PATCH 3/3] - что-то не так с git send-email? Или была отправлена серия
патчей? В списке рассылки вижу только один патч.

Название патча не принято оформлять таким образом: можете посмотреть
командой `git log drivers/gpu/drm/nouveau/nvkm/subdev/mxm/base.c`, какой
обычно добавляют префикс подсистемы - например, "drm/nouveau/mxm".

"BUFFER_OVERFLOW" также совсем не поясняет, что конкретно делает патч.
Обычно пишут что-то вида "fix buffer overflow in <function_name>"

On Thu, 29. Feb 16:10, Andrey Shumilin wrote:
> The mxms_structlen function returns u16 (2 bytes).
> Therefore it reads 2 bytes beyond the mxms array.

Из описания патча непонятно, в чем заключается обнаруженная ошибка
переполнения буфера и каким образом патч ее фиксит.

> 
> Signed-off-by: Andrey Shumilin <shum.sdl at nppct.ru>
> Found by Linux Verification Center (linuxtesting.org) with SVACE.
> ---
>  drivers/gpu/drm/nouveau/nvkm/subdev/mxm/base.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/mxm/base.c b/drivers/gpu/drm/nouveau/nvkm/subdev/mxm/base.c
> index c1acfe642da3..efd0c874742e 100644
> --- a/drivers/gpu/drm/nouveau/nvkm/subdev/mxm/base.c
> +++ b/drivers/gpu/drm/nouveau/nvkm/subdev/mxm/base.c
> @@ -47,7 +47,7 @@ mxm_shadow_rom(struct nvkm_mxm *mxm, u8 version)
>  	struct nvkm_bios *bios = device->bios;
>  	struct nvkm_i2c *i2c = device->i2c;
>  	struct nvkm_i2c_bus *bus = NULL;
> -	u8 i2cidx, mxms[6], addr, size;
> +	u8 i2cidx, mxms[8], addr, size;
>  
>  	i2cidx = mxm_ddc_map(bios, 1 /* LVDS_DDC */) & 0x0f;
>  	if (i2cidx < 0x0f)
> -- 
> 2.30.2
>