On Wed, Oct 06, 2010 at 03:39:41PM -0700, Kees Cook wrote: > Found via http://coccinelle.lip6.fr/rules/mini_null_ref.cocci > > Signed-off-by: Kees Cook <kees.cook@xxxxxxxxxxxxx> > --- > drivers/staging/tm6000/tm6000-alsa.c | 3 ++- > 1 files changed, 2 insertions(+), 1 deletions(-) > > diff --git a/drivers/staging/tm6000/tm6000-alsa.c b/drivers/staging/tm6000/tm6000-alsa.c > index 087137d..c1aff0e 100644 > --- a/drivers/staging/tm6000/tm6000-alsa.c > +++ b/drivers/staging/tm6000/tm6000-alsa.c > @@ -426,10 +426,11 @@ error: > > static int tm6000_audio_fini(struct tm6000_core *dev) > { > - struct snd_tm6000_card *chip = dev->adev; > + struct snd_tm6000_card *chip; > > if (!dev) > return 0; > + chip = dev->adev; Nice catch, but I don't think that the dev pointer can ever be NULL here. Mauro, can you queue this up through your tree, as you handle this one? thanks, greg k-h _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
