On Thu, Nov 23, 2017 at 04:06:28PM +0100, Oliver Neukum wrote: > Am Donnerstag, den 23.11.2017, 15:57 +0100 schrieb Greg KH: > > On Thu, Nov 23, 2017 at 03:28:34PM +0100, Oliver Neukum wrote: > > > > > > USBDEVFS_URB_ISO_ASAP must be accepted only for ISO endpoints. > > > Improve sanity checking. > > > > > > Signed-off-by: Oliver Neukum <oneukum@xxxxxxxx> > > > --- > > > drivers/usb/core/devio.c | 16 ++++++++++------ > > > 1 file changed, 10 insertions(+), 6 deletions(-) > > > > Didn't we do something like this already? > > I sent a patch. Alan found it insufficient. Ah, ok. > > Or was it something else? Was this triggered by some testing? > > Alexey's syzkaller tests showed that you can trigger it. Then a nice "Reported-by:" would be good to add, as would a stable tree inclusion, right? thanks, greg k-h