ALSA: aaci - Fix NULL test at error path
authorTakashi Iwai <tiwai@suse.de>
Mon, 1 Sep 2008 13:31:50 +0000 (15:31 +0200)
committerJaroslav Kysela <perex@perex.cz>
Thu, 4 Sep 2008 08:33:53 +0000 (10:33 +0200)
The original fix by Julien Brunel <brunel@diku.dk>.

aaci_init_card() returns a pointer with ERR_PTR(), but in aaci_init()
NULL is supposed at this error path.

Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Jaroslav Kysela <perex@perex.cz>
sound/arm/aaci.c

index b0a4744..89096e8 100644 (file)
@@ -999,7 +999,7 @@ static struct aaci * __devinit aaci_init_card(struct amba_device *dev)
        card = snd_card_new(SNDRV_DEFAULT_IDX1, SNDRV_DEFAULT_STR1,
                            THIS_MODULE, sizeof(struct aaci));
        if (card == NULL)
-               return ERR_PTR(-ENOMEM);
+               return NULL;
 
        card->private_free = aaci_free_card;
 
@@ -1083,8 +1083,8 @@ static int __devinit aaci_probe(struct amba_device *dev, void *id)
                return ret;
 
        aaci = aaci_init_card(dev);
-       if (IS_ERR(aaci)) {
-               ret = PTR_ERR(aaci);
+       if (!aaci) {
+               ret = -ENOMEM;
                goto out;
        }