Re: [PATCH][RFC] USB: allow malformed LANGID descriptors

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Mar 20, 2009 at 10:34:55AM +0100, Daniel Mack wrote:
> On Thu, Mar 19, 2009 at 08:32:21PM -0700, Greg KH wrote:
> > On Tue, Mar 17, 2009 at 03:52:17PM +0100, Daniel Mack wrote:
> > > When an USB hardware does not provide a valid LANGID, fall back to value
> > > zero which is still a reasonable default for most devices.
> > 
> > But that's a broken device, right?  What happens without this patch?
> 
> It is broken, yes. Without the patch, both the vendor name and the
> product name are swallowed away in kernel space and lsusb.

So it doesn't show up at all anymore?  Do you have a specific example of
this?

> > And has this device passed the usb.org test suite?
> 
> Well, the thing is - this bug was always hidden from us because
> 
> a) Windows doesn't care
> b) Mac OS X doesn't care
> c) Linux used to not care, too. Don't know when that was changed, but it
>    must have been in the last months. So it's kind of a regression, even
>    tough the current behaviour is just currect in terms of the USB spec.

Can you do a little bit of bisection of different kernel versions to
figure out where this changed?

> Anyway, with this patch applied, things work fine and I see no reason
> why complaining with a warning shouldn't be enough. Failing so hard is
> annoying.

True.  If you let me know when this broke, feel free to resend the patch
with that information and I'll be glad to apply it.  I need to know when
it broke so that we can backport it to older -stable kernel versions if
needed as well to make them work again.

thanks,

greg k-h
--
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

[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux