usb: gadget: u_uac1: NULL dereference on error path
authorDan Carpenter <dan.carpenter@oracle.com>
Thu, 28 Feb 2013 04:44:38 +0000 (07:44 +0300)
committerFelipe Balbi <balbi@ti.com>
Mon, 4 Mar 2013 11:16:45 +0000 (13:16 +0200)
We should return here with an error code instead of continuing.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Felipe Balbi <balbi@ti.com>
drivers/usb/gadget/u_uac1.c

index e0c5e88..c7d460f 100644 (file)
@@ -240,8 +240,11 @@ static int gaudio_open_snd_dev(struct gaudio *card)
        snd = &card->playback;
        snd->filp = filp_open(fn_play, O_WRONLY, 0);
        if (IS_ERR(snd->filp)) {
+               int ret = PTR_ERR(snd->filp);
+
                ERROR(card, "No such PCM playback device: %s\n", fn_play);
                snd->filp = NULL;
+               return ret;
        }
        pcm_file = snd->filp->private_data;
        snd->substream = pcm_file->substream;