[lvc-project] [PATCH] drm/vmwgfx: Handle surface check failure correctly

[Fedor Pchelkin]

Николай, здравствуйте!

On Wed, 02. Oct 15:24, Nikolay Kuratov wrote:
> Currently if condition (!bo and !vmw_kms_srf_ok()) was met
> we go to err_out with ret == 0.
> err_out dereferences vfb if ret == 0, but in our case vfb is still NULL.

Думаю, лучше чуть переформулировать абзац выше, чтобы он явно содержал
фразу "разыменовывание нулевого указателя". Также не выдержана длина строк
описания. Все они сильно разной длины. В тексте патча строки допускаются
длиной до 75 символов.

> 
> Fix this by assigning sensible error to ret.
> 
> Found by Linux Verification Center (linuxtesting.org) with SVACE
> 
> Signed-off-by: Nikolay Kuratov <kniv at yandex-team.ru>
> Cc: stable at vger.kernel.org
> Fixes: 810b3e1683d0 ("drm/vmwgfx: Support topology greater than texture size")

Signed-off-by следует ставить после Cc:stable и Fixes. У некоторых
мэйнтейнеров и подсистем на это завязаны автоматические чекеры.

$ ./scripts/get_maintainer.pl -f  drivers/gpu/drm/vmwgfx/vmwgfx_kms.c                              
Zack Rusin <zack.rusin at broadcom.com> (supporter:DRM DRIVER FOR VMWARE VIRTUAL GPU)
Broadcom internal kernel review list <bcm-kernel-feedback-list at broadcom.com> (reviewer:DRM DRIVER FOR VMWARE VIRTUAL GPU)
Maarten Lankhorst <maarten.lankhorst at linux.intel.com> (maintainer:DRM DRIVERS AND MISC GPU PATCHES)
Maxime Ripard <mripard at kernel.org> (maintainer:DRM DRIVERS AND MISC GPU PATCHES)
Thomas Zimmermann <tzimmermann at suse.de> (maintainer:DRM DRIVERS AND MISC GPU PATCHES)
David Airlie <airlied at gmail.com> (maintainer:DRM DRIVERS)
Simona Vetter <simona at ffwll.ch> (maintainer:DRM DRIVERS)
dri-devel at lists.freedesktop.org (open list:DRM DRIVER FOR VMWARE VIRTUAL GPU)
linux-kernel at vger.kernel.org (open list)

Предлагаю отправить v2 с подправленным описанием, выбрав в качестве
основных адресатов (To:) Zack Rusin и Broadcom review list из списка выше.
Остальных поставить в Cc:, в том числе lvc-project at linuxtesting.org.
Отправлять вторую версию стоит новым отдельным mail-тредом, без
In-Reply-To.

Судя по Fixes тегу, баг присутствует в 5.10 и 6.1. Как отправите v2 в
международное сообщество, мы его оттуда заберём без ожидания ответа
мэйнтейнеров.

Спасибо!

> ---
>  drivers/gpu/drm/vmwgfx/vmwgfx_kms.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c b/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c
> index 288ed0bb75cb..752510a11e1b 100644
> --- a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c
> +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c
> @@ -1539,6 +1539,7 @@ static struct drm_framebuffer *vmw_kms_fb_create(struct drm_device *dev,
>  		DRM_ERROR("Surface size cannot exceed %dx%d\n",
>  			dev_priv->texture_max_width,
>  			dev_priv->texture_max_height);
> +		ret = -EINVAL;
>  		goto err_out;
>  	}
>  
> -- 
> 2.34.1