Hi,
this is a patchset to cover the codes that may submit URBs containing
invalid EPs without validation, which result in the kernel warning
from the USB core. The first patch adds a new helper for simplifying
the EP check, and the rest applies it at needed places.
USB devs: could you check the first patch? I tried
usb_find_common_endpoints() and friends at first, but it made the code
more complex in the end, because we're dealing with the fixed
endpoints and the validation is required for them.
The original issues were spotted by syzkaller, and I put a few others
for covering more similar cases.
thanks,
Takashi
===
Takashi Iwai (9):
usb: core: Add a helper function to check the validity of EP type in
URB
ALSA: bcd2000: Add a sanity check for invalid EPs
ALSA: caiaq: Add a sanity check for invalid EPs
ALSA: line6: Add a sanity check for invalid EPs
ALSA: usb-audio: Add sanity checks for invalid EPs
ALSA: usx2y: Add sanity checks for invalid EPs
ALSA: hiface: Add sanity checks for invalid EPs
ALSA: caiaq: Add yet more sanity checks for invalid EPs
ALSA: line6: Add yet more sanity checks for invalid EPs
drivers/usb/core/urb.c | 28 +++++++++++++++++++++++++---
include/linux/usb.h | 2 ++
sound/usb/bcd2000/bcd2000.c | 7 +++++++
sound/usb/caiaq/device.c | 7 +++++++
sound/usb/caiaq/input.c | 9 +++++++++
sound/usb/hiface/pcm.c | 9 +++++++--
sound/usb/line6/driver.c | 30 ++++++++++++++++++++++--------
sound/usb/line6/midi.c | 17 +++++++++++------
sound/usb/midi.c | 38 ++++++++++++++++++++++++++++++--------
sound/usb/usx2y/usbusx2y.c | 8 ++++++++
sound/usb/usx2y/usbusx2yaudio.c | 3 +++
11 files changed, 131 insertions(+), 27 deletions(-)
--
2.14.2
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
