[lvc-project] [PATCH v2] media: usb: hackrf: fix device unregister order in hackrf_probe()

[Жандарович Никита Игоревич]

>-----Original Message-----
>From: Fedor Pchelkin [mailto:pchelkin at ispras.ru]
>Sent: Monday, February 17, 2025 9:30 PM
>To: Жандарович Никита Игоревич <n.zhandarovich at fintech.ru>
>Cc: lvc-project at linuxtesting.org
>Subject: Re: [lvc-project] [PATCH v2] media: usb: hackrf: fix device
>unregister order in hackrf_probe()
>
>On Mon, 17. Feb 16:29, Жандарович Никита Игоревич wrote:
>> Добрый вечер, Фёдор.
>>
>> 100% уверенности в корректности решения у меня нет. Но после
>анализа, локальных экспериментов и проверки syzbot-ом я
>посчитал свой вывод достаточным, чтобы отправить патч.
>
>Ну, в описании не сказано, почему порядок разрегистрации на пути
>ошибки probe-функции сделан неправильно и как это связано с
>репортом KASAN. Я бы не сказал, что это очевидно читателю патча.
>Скорее всего, в её отсутствие в описании патча ему придётся
>дополнительно лезть в исходные тексты ядра и самостоятельно
>выискивать нужную информацию.
>

Согласен, на момент оформления патча этот момент мне казался более очевидным, чем следовало.
Постараюсь быть чуть более подробным в описании проблемы и ее решения.

>*Почему* проблема обозначена связанной с неправильным
>порядком вызова функций? - это вопрос довольно естественно
>появляется, по крайней мере у меня.
>
>> Я также сомневаюсь, что патч в принципе заметят, не то что
>примут.
>
>В media реагируют долго, но реагируют. Ваши патчи туда тоже уже
>принимали.
>Не думаю, что это является каким-то аргументом.