]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
ALSA: sh: Use more common error handling code in snd_aica_probe()
authorMarkus Elfring <elfring@users.sourceforge.net>
Wed, 17 Jun 2026 06:42:18 +0000 (08:42 +0200)
committerTakashi Iwai <tiwai@suse.de>
Wed, 17 Jun 2026 09:06:09 +0000 (11:06 +0200)
Use an additional label so that a bit of exception handling can be better
reused at the end of this function implementation.

This issue was detected by using the Coccinelle software.

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Link: https://patch.msgid.link/47413de1-015b-4543-8e8c-25e41dfa9e39@web.de
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/sh/aica.c

index 9438c3a68ee9ef83809eb504a8844c2908a0e644..8196e1bf041644fe54b07d3b49e2ac3f3763d309 100644 (file)
@@ -564,10 +564,9 @@ static int snd_aica_probe(struct platform_device *devptr)
                return -ENOMEM;
        err = snd_card_new(&devptr->dev, index, SND_AICA_DRIVER,
                           THIS_MODULE, 0, &dreamcastcard->card);
-       if (unlikely(err < 0)) {
-               kfree(dreamcastcard);
-               return err;
-       }
+       if (unlikely(err < 0))
+               goto free_card;
+
        strscpy(dreamcastcard->card->driver, "snd_aica");
        strscpy(dreamcastcard->card->shortname, SND_AICA_DRIVER);
        strscpy(dreamcastcard->card->longname,
@@ -593,6 +592,7 @@ static int snd_aica_probe(struct platform_device *devptr)
        return 0;
       freedreamcast:
        snd_card_free(dreamcastcard->card);
+free_card:
        kfree(dreamcastcard);
        return err;
 }