[lvc-project] [PATCH 5.10] Bluetooth: btmtksdio: fix use-after-free at btmtksdio_recv_event

[Anastasia Belova]

21/07/25 18:13, Fedor Pchelkin пишет:
> On Mon, 21. Jul 17:14, Anastasia Belova wrote:
>>> Таким образом, в evt на стеке может в этом случае храниться устаревшее
>>> значение, с которым потом идёт сравнение.
>>>
>>> Поэтому важно свою проводимую нетривиальную в большинстве случаев
>>> адаптацию описывать: удобнее всего либо в комментарии бэкпортёра в
>>> квадратных скобках над своей подписью (пойдет в гит-историю), либо в
>>> зоне после ---. Если портируется серия патчей, то можно свои мысли
>>> изложить в сопроводительном письме.
>>>
>>> Это позволит другим людям составить представление о том, какие
>>> дополнительные действия проводились по адаптации патча: важно, чтобы они
>>> были сделаны не машинально на уровне "применилось/скомпилировалось, а
>>> большего и не надо", а всё-таки был изучен контекст патча, суть конфликта
>>> и т.п.
>>>
>>> Все нюансы отражены в
>>> https://portal.linuxtesting.ru/How-to-send-patches-to-kernel.html#Сопроводительное-письмо
>>>
>>>
>>> Конфликт в контексте 5.10 растёт из-за отсутствия там коммита
>>>
>>> commit e4412654e260842e1a94ffe0d4026e8a6fd34246
>>> Author: Sean Wang <sean.wang at mediatek.com>
>>> Date:   Wed Feb 9 02:17:41 2022 +0800
>>>
>>>       Bluetooth: mediatek: fix the conflict between mtk and msft vendor event
>>>
>>>
>>> Можно ознакомиться с его содержимым и основной целью. Не вижу серьёзных
>>> причин не портировать его в 5.10.
>> Этот бэкпорт повторяет бэкпорт b3cec8a42fcd11d05313c724f27e01b1db77522c для
>> 5.15.
> Не стоит повторять ошибки, которые могут быть сделаны при автоматическом
> портировании патчей скриптами и нейросетями stable-мэйнтейнеров. Необходимо
> самостоятельно оценивать корректность переносимых в целевую ветку патчей
> и стараться не добавлять новых (хотя и менее значительных) ошибок.
>
> А ошибка есть, и она связана с неправильным учётом контекста стабильной
> ветки - детали я описал выше.
>
>> Туда e4412654e260842e1a94ffe0d4026e8a6fd34246 не был затянут. Предлагаю все
>> же
>> для соответствия с международной веткой 5.15 оставить адаптацию такой. Если
>> комментарий к разрешению конфликта нужен обязательно, могу сформировать
>> вторую версию.
> Здесь дело обстоит таким образом, что в 5.15 появилась дополнительная
> ошибка, и для её подавления туда нужно дополнительно портировать
> e4412654e260842e1a94ffe0d4026e8a6fd34246.
>
> Соответственно для 5.10 этот доп. коммит тоже требуется, раз уж туда
> переносится "Bluetooth: btmtksdio: fix use-after-free at btmtksdio_recv_event".
>
> Центр не сопровождает ветку 5.15, и учёт того, как обстоят дела там,
> остаётся на личное усмотрение Участников.
>
> Если имеется желание/возможность доводить бэкпорты до межд. сообщества,
> то необходимо учитывать все более свежие поддерживаемые стабильные ветки
> (5.15, в частности).
>
> Для закрытия задач в рамках работ исправления должны содержаться в 5.10
> и 6.1 ветках Центра. Можно направлять сразу через lvc-patches at linuxtesting.org.

Немного искаженно сначала восприняла первое письмо, сейчас стало понятно,
что за ошибка. Коммит e4412654e260842e1a94ffe0d4026e8a6fd34246 связан с
еще одним отсутствующим в 5.10 коммитом 
145373cb1b1fcdba2059e945d0aa2613af2e84d1,
вносящим новую функциональность. В общем можно бэкпортировать так, чтобы его
не затягивать, но может лучше перенести присваивание в evt ниже при 
адаптации
изначально бекпортируемого коммита?