<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=koi8-r">
<meta name="Generator" content="Microsoft Exchange Server">
<!-- converted from text --><style><!-- .EmailQuote { margin-left: 1pt; padding-left: 4pt; border-left: #800000 2px solid; } --></style>
</head>
<body>
<meta content="text/html; charset=UTF-8">
<style type="text/css" style="">
<!--
p
{margin-top:0;
margin-bottom:0}
-->
</style>
<div dir="ltr">
<div id="x_divtagdefaultwrapper" dir="ltr" style="font-size:12pt; color:#000000; font-family:Calibri,Helvetica,sans-serif">
<p>Особой причины нет - п<span>еременная `int err` уже была смещена в исходнике ядра на устройство. Так и попала в бэкпорт-версию.</span></p>
<p>Исправить не проблема. Вышлю патч.</p>
<p><br>
</p>
<p>Освоить <span>git send-email не проблема. В этом патче моя ошибка - поставила `To` вместо `Cc`.<span></span></span><br>
</p>
</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> 3 июня 2026 г. 19:06:45<br>
<b>Кому:</b> Viktoriya Danchenko<br>
<b>Копия:</b> lvc-project@linuxtesting.org<br>
<b>Тема:</b> Re: [lvc-project] [PATCH 5.10] crypto: lzo - Fix compression buffer overrun</font>
<div> </div>
</div>
</div>
<font size="2"><span style="font-size:10pt;">
<div class="PlainText">Письмо отправлено внешним отправителем. Не переходите по ссылкам и не открывайте вложения, если не уверены в их безопасности.<br>
<br>
On Wed, 03. Jun 12:42, Viktoriya Danchenko wrote:<br>
> From: Herbert Xu <herbert@gondor.apana.org.au><br>
><br>
> [ Upstream commit cc47f07234f72cbd8e2c973cdbf2a6730660a463 ]<br>
<br>
Не забывайте, git send-email любит ставить в копию автоматических<br>
адресатов из тела патча, которым не всё следует отправлять. См.<br>
<a href="https://portal.linuxtesting.ru/How-to-send-patches-to-kernel.html#Отправка">https://portal.linuxtesting.ru/How-to-send-patches-to-kernel.html#Отправка</a><br>
<br>
> @@ -312,6 +347,7 @@ int lzogeneric1x_1_compress(const unsigned char *in, size_t in_len,<br>
> size_t t = 0;<br>
> signed char state_offset = -2;<br>
> unsigned int m4_max_offset;<br>
> + int err;<br>
<br>
Есть какая-то причина, почему переменная `int err` в бэкпорт-версии<br>
определяется для всей функции, а не внутри цикла while на пару десятков<br>
строк ниже, как делает апстримный коммит?<br>
<br>
Там где возможно, лучше не отступать от апстрима - здесь не совсем<br>
понятно, зачем это было произведено.<br>
<br>
><br>
> // LZO v0 will never write 17 as first byte (except for zero-length<br>
> // input), so this is used to version the bitstream<br>
> @@ -332,10 +368,12 @@ int lzogeneric1x_1_compress(const unsigned char *in, size_t in_len,<br>
> break;<br>
> BUILD_BUG_ON(D_SIZE * sizeof(lzo_dict_t) > LZO1X_1_MEM_COMPRESS);<br>
> memset(wrkmem, 0, D_SIZE * sizeof(lzo_dict_t));<br>
> - t = lzo1x_1_do_compress(ip, ll, op, out_len, t, wrkmem,<br>
> - &state_offset, bitstream_version);<br>
> + err = LZO_SAFE(lzo1x_1_do_compress)(<br>
> + ip, ll, &op, op_end, &t, wrkmem,<br>
> + &state_offset, bitstream_version);<br>
> + if (err != LZO_E_OK)<br>
> + return err;<br>
> ip += ll;<br>
> - op += *out_len;<br>
> l -= ll;<br>
> }<br>
> t += l;<br>
</div>
</span></font>
</body>
</html>