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 <[email protected]>
> 
> ---
> 
> 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 [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to