On Thu, Jan 24, 2013 at 03:04:13PM -0500, Alan Stern wrote: > This patch (as1646) fixes a long-standing bug in the USB hub driver. > Upon conversion from char to unsigned long, the bytes in the status > buffer are subject to unwanted sign extension. The bytes should be > declared as u8 rather than char, to prevent this. > > This effects of this bug are minimal. The hub driver may end up doing > a little unnecessary extra work because it thinks events have occurred > on some ports when they really haven't. > > Signed-off-by: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> > > --- > > Greg: > > Although it is technically a bug, the effects caused by this error are > so minor that I don't know whether it's worth queuing this for the > stable kernels. I'll leave that decision up to you. I'll just leave it alone and let this patch go into 3.9. 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
