[lvc-project] Патч для рецнзии

[Alexey Khoroshilov]

Добрый вечер!

On 27.02.2023 14:50, Данила Чернецов wrote:
>
> Добрый день!
> В рамках работ по статистическому анализу кода ядра linux-5.10.yперед
> отправкой в upstream, согласно рекомендациям, направляем вам на
> согласование патч по срабатыванию NULL_AFTER_DEREF:
> drivers/gpu/drm/amd/display/dc/core/dc.c:1411
> <https://lvc-svacer.ispras.ru/mode/review/project/83b681dd-c2b5-459d-80cc-359ff2a64935/branch/4739abc2-5b35-494c-9748-062208797cd3/snapshot/952a1137-9415-42e4-8f91-83c6169ede53/marker/eyJtYXJrZXJJRCI6ICIxOTlkMTk0NS0xMTgzLTQyMmItOWY0Ni1iM2QxOTgwMjQyZmYiLCAiZmlsZSI6ICIvZHJpdmVycy9ncHUvZHJtL2FtZC9kaXNwbGF5L2RjL2NvcmUvZGMuYyJ9>Просим
> вас провести ревью и указать возможные недочеты.
> Во вложении патчи для upstream 
> (0002-drm-amd-display-remove-an-unnecessary-NULL-check.patch) и
> linux-5.10.y
> (0003-drm-amd-display-remove-an-unnecessary-NULL-check.patch).
>
1. Патч для linux-5.10.y потребуется после того как будет принят патч в
upstream.
2.
Subject: [PATCH 2/2] drm/amd/display: remove an unnecessary NULL check
-->
Subject: [PATCH] drm/amd/display: remove an unnecessary NULL check

3.
The "pipe" pointer can't be NULL because it's dereferenced on the line
before.
не совсем корректно, поскольку на предыдущей строке не разыменование
указателя,
 а взятие указателя на элемент массива, являющегося полем структуры.
Скорее:
The 'pipe' can't be NULL because it points to an element of array.

4. И хотя некоторые мейнтейнеры не соглашаются с Fixes: при удалении
лишнего кода, предлагаю добавить

Fixes: 7cf2c840c6b5 ("drm/amd/display: Commit validation set from state")


Алексей
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://linuxtesting.org/pipermail/lvc-project/attachments/20230227/5c81a94c/attachment.html>