usb_find_alt_setting() takes a pointer to a struct usb_host_config as
an argument; it searches for an interface with specified interface and
alternate setting numbers in that config. However, it crashes if the
usb_host_config pointer argument is NULL.
Since this is a general-purpose routine, available for use in many
places, we want to to be more robust. This patch makes it return NULL
whenever the config argument is NULL.
Signed-off-by: Alan Stern <st...@rowland.harvard.edu>
Reported-by: syzbot+19c3aaef85a89d451...@syzkaller.appspotmail.com
CC: <sta...@vger.kernel.org>
---
[as1879]
drivers/usb/core/usb.c | 2 ++
1 file changed, 2 insertions(+)
Index: usb-4.x/drivers/usb/core/usb.c
===================================================================
--- usb-4.x.orig/drivers/usb/core/usb.c
+++ usb-4.x/drivers/usb/core/usb.c
@@ -228,6 +228,8 @@ struct usb_host_interface *usb_find_alt_
struct usb_interface_cache *intf_cache = NULL;
int i;
+ if (!config)
+ return NULL;
for (i = 0; i < config->desc.bNumInterfaces; i++) {
if (config->intf_cache[i]->altsetting[0].desc.bInterfaceNumber
== iface_num) {
