]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
ALSA: scarlett2: Add missing error check when initialise Autogain Status
authorRobertus Diawan Chris <robertusdchris@gmail.com>
Fri, 8 May 2026 03:39:14 +0000 (10:39 +0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 1 Jun 2026 15:46:30 +0000 (17:46 +0200)
[ Upstream commit c0e4fffc0f474b7ed10adee4ab2bc1a66d36fc72 ]

When initialise new control with scarlett2_add_new_ctl() function for
Autogain Status, scarlett2_add_new_ctl() might throw an error. So, add
error check after initialise new control for Autogain Status.

This is reported by Coverity Scan with CID 1598781 as UNUSED_VALUE.

Fixes: 0a995e38dc44 ("ALSA: scarlett2: Add support for software-controllable input gain")
Signed-off-by: Robertus Diawan Chris <robertusdchris@gmail.com>
Link: https://patch.msgid.link/20260508033914.111596-1-robertusdchris@gmail.com
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Sasha Levin <sashal@kernel.org>
sound/usb/mixer_scarlett2.c

index ef5945aa40e4aa4bb7057be67ccbbd75ce9649b3..d767a89e452d17fd07df19bf3d989d8f773b17f6 100644 (file)
@@ -6956,6 +6956,8 @@ static int scarlett2_add_line_in_ctls(struct usb_mixer_interface *mixer)
                err = scarlett2_add_new_ctl(
                        mixer, &scarlett2_autogain_status_ctl,
                        i, 1, s, &private->autogain_status_ctls[i]);
+               if (err < 0)
+                       return err;
        }
 
        /* Add autogain target controls */