Re: [v4.9-rc4] dvb-usb/cinergyT2 NULL pointer dereference

Mauro Carvalho Chehab
2016-11-09T20:25:32Z

Em Wed, 9 Nov 2016 19:57:58 +0000
Malcolm Priestley <tvboxspy@gmail.com> escreveu:

> > Yeah, I avoided serializing the logic that detects if the firmware is
> > loaded, but forgot that the power control had the same issue. The
> > newer dvb usb drivers use the dvb-usb-v2, so I didn't touch this
> > code for a while.
>
> I think the problem is that the usb buffer has been put in struct
> cinergyt2_state private area which has not been initialized for initial
> usb probing.
>
> That was one of the main reasons for porting drivers to dvb-usb-v2.

True, but converting to dvb-usb-v2, is more complex. In particular,
dib0700 and dib3000 drivers rely on some things that may not be ported
to dvb-usb-v2.

So, I don't think we should do such change during the -rc cycle.
Also, such change requires testing. So, one with those hardware should
help with it, or the developer willing to do the conversion would
need to get those old hardware in hands.

Thanks,
Mauro