[lvc-project] [PATCH] media: go7007: add check of return value of go7007_read_addr()

[Fedor Pchelkin]

On Fri, 15. Mar 12:45, Fedor Pchelkin wrote:
> On Sun, 11. Feb 07:07, Daniil Dulov wrote:
> > If go7007_read_addr() returns error channel is not assigned a value.
> > In this case go to allocfail.
> > 
> > Found by Linux Verification Center (linuxtesting.org) with SVACE.
> > 
> > Fixes: 866b8695d67e ("Staging: add the go7007 video driver")
> > Signed-off-by: Daniil Dulov <d.dulov at aladdin.ru>
> > ---
> >  drivers/media/usb/go7007/go7007-usb.c | 4 +++-
> >  1 file changed, 3 insertions(+), 1 deletion(-)
> > 
> > diff --git a/drivers/media/usb/go7007/go7007-usb.c b/drivers/media/usb/go7007/go7007-usb.c
> > index eeb85981e02b..762c13e49bfa 100644
> > --- a/drivers/media/usb/go7007/go7007-usb.c
> > +++ b/drivers/media/usb/go7007/go7007-usb.c
> > @@ -1201,7 +1201,9 @@ static int go7007_usb_probe(struct usb_interface *intf,
> >  				u16 channel;
> >  
> >  				/* read channel number from GPIO[1:0] */
> > -				go7007_read_addr(go, 0x3c81, &channel);
> > +				if (go7007_read_addr(go, 0x3c81, &channel))
> 
> Добрый день!
> 
> Стоит ли здесь вывести поясняющее сообщение об ошибке, как сделано во всех
> других ошибочных ситуациях в probe функции этого драйвера?
> 

В любом случае, смерджено в upstream [1].

Добавим в ветки linux-5.10-lvc-next и linux-6.1-lvc-next.

Спасибо!

[1]: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=0b70530ee740861f4776ff724fcc25023df1799a