usb: gadget: f_uac2: Fix incorrect setting of bNumEndpoints

Prashanth K · mehmetb0 · commit 72635fef1481 · 2025-04-15T19:59:46.000+03:00
BugLink: https://bugs.launchpad.net/bugs/2106770 commit 057bd54 upstream. Currently afunc_bind sets std_ac_if_desc.bNumEndpoints to 1 if controls (mute/volume) are enabled. During next afunc_bind call, bNumEndpoints would be unchanged and incorrectly set to 1 even if the controls aren't enabled. Fix this by resetting the value of bNumEndpoints to 0 on every afunc_bind call. Fixes: eaf6cbe ("usb: gadget: f_uac2: add volume and mute support") Cc: stable <stable@kernel.org> Signed-off-by: Prashanth K <quic_prashk@quicinc.com> Link: https://lore.kernel.org/r/20241211115915.159864-1-quic_prashk@quicinc.com Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Signed-off-by: Manuel Diewald <manuel.diewald@canonical.com> Signed-off-by: Mehmet Basaran <mehmet.basaran@canonical.com>
diff --git a/drivers/usb/gadget/function/f_uac2.c b/drivers/usb/gadget/function/f_uac2.c
@@ -1179,6 +1179,7 @@ afunc_bind(struct usb_configuration *cfg, struct usb_function *fn)
 		uac2->as_in_alt = 0;
 	}
 
+	std_ac_if_desc.bNumEndpoints = 0;
 	if (FUOUT_EN(uac2_opts) || FUIN_EN(uac2_opts)) {
 		uac2->int_ep = usb_ep_autoconfig(gadget, &fs_ep_int_desc);
 		if (!uac2->int_ep) {

Original file line number	Diff line number	Diff line change
`@@ -1179,6 +1179,7 @@ afunc_bind(struct usb_configuration cfg, struct usb_function fn)`
`1179`	`1179`	`uac2->as_in_alt = 0;`
`1180`	`1180`	`}`
`1181`	`1181`
	`1182`	`+ std_ac_if_desc.bNumEndpoints = 0;`
`1182`	`1183`	`if (FUOUT_EN(uac2_opts) \|\| FUIN_EN(uac2_opts)) {`
`1183`	`1184`	`uac2->int_ep = usb_ep_autoconfig(gadget, &fs_ep_int_desc);`
`1184`	`1185`	`if (!uac2->int_ep) {`