<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=koi8-r">
<style type="text/css" style="display:none;"><!-- P {margin-top:0;margin-bottom:0;} --></style>
</head>
<body dir="ltr">
<div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Helvetica,sans-serif;" dir="ltr">
<div id="divtagdefaultwrapper" style="font-size: 12pt; color: rgb(0, 0, 0); font-family: Calibri, Helvetica, sans-serif, "EmojiFont", "Apple Color Emoji", "Segoe UI Emoji", NotoColorEmoji, "Segoe UI Symbol", "Android Emoji", EmojiSymbols;" dir="ltr">
<p>Здравствуйте, Федор!<br>
</p>
<p><br>
</p>
<p>Это был самый первый мой патч, так что, как ни старался, без косяков, увы, не обошлось. В последующих постепенно стараюсь их избегать. Большое спасибо за замечания.</p>
<p>Конкретно по пунктам:</p>
<ol style="margin-bottom: 0px; margin-top: 0px;">
<li>По названию - честно говоря, я свой вариант перевел как "добавление проверки на разыменование нулевого указателя". Но с Вашим вариантом согласен, он более полно отражает проблему, поменяю.</li><li>Про пустые строки Алексей мне уже написал, в последующих патчах (кроме одного) я это учел.</li><li>По коду возврата - согласен, не учел, исправлю.</li></ol>
<div><br>
</div>
<div>Кроме того, в нескольких последующих патчах имеется ошибка с потерей кавычек в теге Fixes, т.е.</div>
<div><span></span></div>
<div><span><span>Fixes: 4a835afd808a (mmc: dw_mmc: Fix potential null pointer risk)</span></span></div>
<div><span>вместо<br>
</span></div>
<div><span></span></div>
<span><span>Fixes: 4a835afd808a ("mmc: dw_mmc: Fix potential null pointer risk")</span></span></div>
<div style="font-size: 12pt; color: rgb(0, 0, 0); font-family: Calibri, Helvetica, sans-serif, "EmojiFont", "Apple Color Emoji", "Segoe UI Emoji", NotoColorEmoji, "Segoe UI Symbol", "Android Emoji", EmojiSymbols;" dir="ltr">
<span><span>Ошибка чисто техническая, но не заметил. Учел.</span></span></div>
<div style="font-size: 12pt; color: rgb(0, 0, 0); font-family: Calibri, Helvetica, sans-serif, "EmojiFont", "Apple Color Emoji", "Segoe UI Emoji", NotoColorEmoji, "Segoe UI Symbol", "Android Emoji", EmojiSymbols;" dir="ltr">
<span><span><br>
</span></span></div>
<div style="font-size: 12pt; color: rgb(0, 0, 0); font-family: Calibri, Helvetica, sans-serif, "EmojiFont", "Apple Color Emoji", "Segoe UI Emoji", NotoColorEmoji, "Segoe UI Symbol", "Android Emoji", EmojiSymbols;" dir="ltr">
<span><span>Касаемо этого патча: я правильно понимаю, что мне необходимо выпустить вторую версию согласно п. "<span>Подготовка второй версии патча</span>" из
<a href="https://portal.linuxtesting.ru/How-to-send-patches-to-kernel.html" class="OWAAutoLink" id="LPlnk565991">
https://portal.linuxtesting.ru/How-to-send-patches-to-kernel.html</a>?<br>
</span></span></div>
<div style="font-size: 12pt; color: rgb(0, 0, 0); font-family: Calibri, Helvetica, sans-serif, "EmojiFont", "Apple Color Emoji", "Segoe UI Emoji", NotoColorEmoji, "Segoe UI Symbol", "Android Emoji", EmojiSymbols;" dir="ltr">
<span><span><br>
</span></span></div>
<div style="font-size: 12pt; color: rgb(0, 0, 0); font-family: Calibri, Helvetica, sans-serif, "EmojiFont", "Apple Color Emoji", "Segoe UI Emoji", NotoColorEmoji, "Segoe UI Symbol", "Android Emoji", EmojiSymbols;" dir="ltr">
С уважением,
<div id="Signature">
<div id="divtagdefaultwrapper" dir="ltr" style="font-size: 12pt; color: rgb(0, 0, 0); font-family: Calibri, Helvetica, sans-serif, "EmojiFont", "Apple Color Emoji", "Segoe UI Emoji", NotoColorEmoji, "Segoe UI Symbol", "Android Emoji", EmojiSymbols;">
<p>Мишин Александр</p>
<p>ООО "ТехАргос"<br>
</p>
</div>
</div>
<br>
<br>
<div style="color: rgb(0, 0, 0);">
<div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="x_divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>От:</b> Fedor Pchelkin <pchelkin@ispras.ru><br>
<b>Отправлено:</b> 15 марта 2024 г. 22:59<br>
<b>Кому:</b> Мишин Александр Борисович<br>
<b>Копия:</b> lvc-project@linuxtesting.org<br>
<b>Тема:</b> Re: [lvc-project] [PATCH] crypto: rockchip/rk3288 - Add dereference of NULL pointer check</font>
<div> </div>
</div>
</div>
<font size="2"><span style="font-size:10pt;">
<div class="PlainText">Здравствуйте, Александр!<br>
<br>
Название патча я перевожу как "Добавить разыменовывание проверки на нулевой<br>
указатель"?) Стоит его назвать по-другому. Что-то вида "Handle crypto<br>
engine allocation failure".<br>
<br>
On Mon, 04. Mar 15:20, Aleksandr Mishin wrote:<br>
> In rk_crypto_probe() crypto_engine_alloc_init() is assigned to<br>
> crypto_info->engine and there is a dereference of it in<br>
> clk_mt2712_top_init_early() which could lead to a NULL pointer<br>
> dereference on failure of crypto_engine_alloc_init().<br>
> <br>
> Fix this bug by adding a check of crypto_info->engine.<br>
> <br>
> Found by Linux Verification Center (linuxtesting.org) with SVACE.<br>
> <br>
> Fixes: 57d67c6e8219 ("crypto: rockchip - rework by using crypto_engine")<br>
> <br>
> Signed-off-by: Aleksandr Mishin <amishin@t-argos.ru><br>
<br>
Пустых строк между тегом Fixes и подписью быть не должно.<br>
<br>
> ---<br>
> drivers/crypto/rockchip/rk3288_crypto.c | 4 ++++<br>
> 1 file changed, 4 insertions(+)<br>
> <br>
> diff --git a/drivers/crypto/rockchip/rk3288_crypto.c b/drivers/crypto/rockchip/rk3288_crypto.c<br>
> index 70edf40bc523..88cea1e36afa 100644<br>
> --- a/drivers/crypto/rockchip/rk3288_crypto.c<br>
> +++ b/drivers/crypto/rockchip/rk3288_crypto.c<br>
> @@ -371,6 +371,10 @@ static int rk_crypto_probe(struct platform_device *pdev)<br>
> }<br>
> <br>
> crypto_info->engine = crypto_engine_alloc_init(&pdev->dev, true);<br>
> + if (!crypto_info->engine) {<br>
> + dev_err(&pdev->dev, "memory allocation failed.\n");<br>
<br>
Стоит присвоить статус ошибки возвращаемому из probe функции значению.<br>
Здесь подойдет -ENOMEM. Т.к. err изначально нулевой, то функция возвратит<br>
ноль и все будут считать, что она завершилась успешно.<br>
<br>
> + goto err_crypto;<br>
> + }<br>
> crypto_engine_start(crypto_info->engine);<br>
> init_completion(&crypto_info->complete);<br>
> <br>
> -- <br>
> 2.30.2<br>
> <br>
> <br>
</div>
</span></font></div>
</div>
</div>
</body>
</html>