Nikita Zhandarovich 6ec3514a7d comedi: vmk80xx: fix incomplete endpoint checking ... commit d1718530e3 upstream. While vmk80xx does have endpoint checking implemented, some things can fall through the cracks. Depending on the hardware model, URBs can have either bulk or interrupt type, and current version of vmk80xx_find_usb_endpoints() function does not take that fully into account. While this warning does not seem to be too harmful, at the very least it will crash systems with 'panic_on_warn' set on them. Fix the issue found by Syzkaller [1] by somewhat simplifying the endpoint checking process with usb_find_common_endpoints() and ensuring that only expected endpoint types are present. This patch has not been tested on real hardware. [1] Syzkaller report: usb 1-1: BOGUS urb xfer, pipe 1 != type 3 WARNING: CPU: 0 PID: 781 at drivers/usb/core/urb.c:504 usb_submit_urb+0xc4e/0x18c0 drivers/usb/core/urb.c:503 ... Call Trace: <TASK> usb_start_wait_urb+0x113/0x520 drivers/usb/core/message.c:59 vmk80xx_reset_device drivers/comedi/drivers/vmk80xx.c:227 [inline] vmk80xx_auto_attach+0xa1c/0x1a40 drivers/comedi/drivers/vmk80xx.c:818 comedi_auto_config+0x238/0x380 drivers/comedi/drivers.c:1067 usb_probe_interface+0x5cd/0xb00 drivers/usb/core/driver.c:399 ... Similar issue also found by Syzkaller: Link: https://syzkaller.appspot.com/bug?extid=5205eb2f17de3e01946e Reported-and-tested-by: syzbot+5f29dc6a889fc42bd896@syzkaller.appspotmail.com Cc: stable <stable@kernel.org> Fixes: 49253d542c ("staging: comedi: vmk80xx: factor out usb endpoint detection") Reviewed-by: Ian Abbott <abbotti@mev.co.uk> Signed-off-by: Nikita Zhandarovich <n.zhandarovich@fintech.ru> Link: https://lore.kernel.org/r/20240408171633.31649-1-n.zhandarovich@fintech.ru Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>		2024-04-27 17:12:58 +02:00
..
drivers	comedi: vmk80xx: fix incomplete endpoint checking	2024-04-27 17:12:58 +02:00
kcomedilib	comedi: Move the main COMEDI headers	2021-11-26 16:48:59 +01:00
comedi_buf.c	comedi: Move the main COMEDI headers	2021-11-26 16:48:59 +01:00
comedi_fops.c	drivers/comedi: use standard array-copy-function	2023-12-07 11:08:45 +09:00
comedi_internal.h
comedi_pci.c	comedi: Move the main COMEDI headers	2021-11-26 16:48:59 +01:00
comedi_pcmcia.c	comedi: Move the main COMEDI headers	2021-11-26 16:48:59 +01:00
comedi_usb.c	comedi: Move the main COMEDI headers	2021-11-26 16:48:59 +01:00
drivers.c	comedi: comedi_8254: Replace comedi_8254_init() and comedi_8254_mm_init()	2023-10-05 13:34:04 +02:00
Kconfig	comedi: add HAS_IOPORT dependencies again	2023-10-05 13:34:05 +02:00
Makefile
proc.c	comedi: Move the main COMEDI headers	2021-11-26 16:48:59 +01:00
range.c	comedi: Move the main COMEDI headers	2021-11-26 16:48:59 +01:00
TODO