On Fri, 17 Aug 2007, Oliver Endriss wrote:
> Markus Rechberger wrote:
> > Since this didn't get commented here, Trent did that patch already 2
> > months ago but it's not included yet. So I recommend to include his
> > patch.
> >
> > http://article.gmane.org/gmane.linux.kernel/543689
> >
> > Acked-by: Markus Rechberger <[EMAIL PROTECTED]>
>
> Are you talking about this tiny patch?
>
>       struct dvb_net *dvbnet = dvbdev->priv;
>
> -     if (!dvbdev)
> -             return -ENODEV;
>
> If yes,
> Acked-by: Oliver Endriss <[EMAIL PROTECTED]>
>
> Btw, why is this patch important? Basically it doesn't change anything.

It checks if dvbdev is NULL after dvbdev already been used.  Coverity spots
this as a programming mistake (which it is), and Adrian Bunk posts patches
to fix it.

_______________________________________________
linux-dvb mailing list
linux-dvb@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb

Reply via email to