[alsa-devel] [patch] ALSA: lola: NULL deref on allocation error

Takashi Iwai tiwai at suse.de
Wed Mar 5 12:11:31 CET 2014


At Wed, 5 Mar 2014 13:55:09 +0300,
Dan Carpenter wrote:
> 
> "chip" is NULL here.  We don't need a printk here because kmalloc() has
> it built in.
> 
> Signed-off-by: Dan Carpenter <dan.carpenter at oracle.com>

Thanks, applied.


Takashi

> 
> diff --git a/sound/pci/lola/lola.c b/sound/pci/lola/lola.c
> index d63434d8c3fb..68824cdd137d 100644
> --- a/sound/pci/lola/lola.c
> +++ b/sound/pci/lola/lola.c
> @@ -586,7 +586,6 @@ static int lola_create(struct snd_card *card, struct pci_dev *pci,
>  
>  	chip = kzalloc(sizeof(*chip), GFP_KERNEL);
>  	if (!chip) {
> -		dev_err(chip->card->dev, "cannot allocate chip\n");
>  		pci_disable_device(pci);
>  		return -ENOMEM;
>  	}
> 


More information about the Alsa-devel mailing list