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

Fedor Pchelkin pchelkin at ispras.ru
Mon Feb 17 21:29:47 MSK 2025


On Mon, 17. Feb 16:29, Жандарович Никита Игоревич wrote:
> Добрый вечер, Фёдор.
> 
> 100% уверенности в корректности решения у меня нет. Но после анализа, локальных экспериментов и проверки syzbot-ом я посчитал свой вывод достаточным, чтобы отправить патч.

Ну, в описании не сказано, почему порядок разрегистрации на пути ошибки
probe-функции сделан неправильно и как это связано с репортом KASAN. Я
бы не сказал, что это очевидно читателю патча. Скорее всего, в её
отсутствие в описании патча ему придётся дополнительно лезть в исходные
тексты ядра и самостоятельно выискивать нужную информацию.

*Почему* проблема обозначена связанной с неправильным порядком вызова
функций? - это вопрос довольно естественно появляется, по крайней мере у
меня.

> Я также сомневаюсь, что патч в принципе заметят, не то что примут. 

В media реагируют долго, но реагируют. Ваши патчи туда тоже уже принимали.
Не думаю, что это является каким-то аргументом.



More information about the lvc-project mailing list