[lvc-project] [PATCH] media: ngene: Add dvb_ca_en50221_init return value check

Fedor Pchelkin pchelkin at ispras.ru
Fri Mar 15 17:16:37 MSK 2024


On Fri, 15. Mar 16:58, Fedor Pchelkin wrote:
> On Fri, 01. Mar 14:15, Aleksandr Burakov wrote:
> > The return value of dvb_ca_en50221_init() is not checked here that may
> > cause undefined behavior in case of nonzero value return.
> > 
> > Found by Linux Verification Center (linuxtesting.org) with SVACE.
> > 
> > Fixes: 25aee3debe04 ("[media] Rename media/dvb as media/pci")
> 
> Добавлено в ветки linux-5.10-lvc-next и linux-6.1-lvc-next.
> 
> Коммит `Fixes` не совсем верный, заменил его на коммит 3d0cef2c3c7e
> ("[media] ngene: Improved channel initialisation and release").
> 
> В публичной linux-5.10-lvc [1] появится при выходе ближайшего релиза.
> 
> Спасибо!
> 
> [1]: https://git.linuxtesting.ru/pub/scm/linux/kernel/git/lvc/linux-stable.git/log/

Мне кажется, в этой функции также неправильно реализована обработка ошибок
в других местах. Например, по метке `err:` возвращается ноль, а не `ret`.
Также, не вызывается, к примеру, dvb_unregister_device() в случае обработок
ошибок в этой функции.

Напишите пожалуйста, желаете ли подготовить патч для этого?

--
Федор



More information about the lvc-project mailing list