On Tue, 2010-01-12 at 12:28 +0100, Jiri Slaby wrote: > On 01/12/2010 12:48 AM, Andy Walls wrote: > > On Sun, 2010-01-10 at 09:56 +0100, Jiri Slaby wrote: > >> Stanse found a potential null dereference in cx18_dvb_start_feed > >> and cx18_dvb_stop_feed. There is a check for stream being NULL, > >> but it is dereferenced earlier. Move the dereference after the > >> check. > >> > >> Signed-off-by: Jiri Slaby <jslaby@xxxxxxx> > > > > Reviewed-by: Andy Walls <awalls@xxxxxxxxx> > > Acked-by: Andy Walls <awalls@xxxxxxxxx> > > You definitely know the code better, have you checked that it can happen > at all? I mean may demux->priv be NULL? I'm wasn't sure, and that's the one reason I didn't NAK the patch. I can tell you no one has ever reported an Ooops or Bug due to that condition. I know the cx18 code very well. However, I am less familiar with the dvb core code and any bad behavior that may exist there. When relying on data structures the dvb core accesses I would have to research what could happen in the dvb core to possibly generate that condition. Since I'm busy this week with work related to my day job (nothing to do with Linux), it was easiest to let the NULL check stay in for now. If you don't mind a delay of until Sunday or so to get the patch applied to the V4L-DVB tree, I can take the patch and work it in my normal path through Mauro. Let me know. Regards, Andy -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
