Greg:

Sorry, the Subject line of the original email left out "[PATCH]" at the 
beginning.  Will it still work with your scripts?

Alan Stern


On Thu, 23 Jun 2016, Alan Stern wrote:

> The HOSTPC extension registers found in some EHCI implementations form
> a variable-length array, with one element for each port.  Therefore
> the hostpc field in struct ehci_regs should be declared as a
> zero-length array, not a single-element array.
> 
> This fixes a problem reported by UBSAN.
> 
> Signed-off-by: Alan Stern <[email protected]>
> Reported-by: Wilfried Klaebe <[email protected]>
> Tested-by: Wilfried Klaebe <[email protected]>
> CC: <[email protected]>
> 
> ---
> 
> 
> [as1803]
> 
> 
>  include/linux/usb/ehci_def.h |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> Index: usb-4.x/include/linux/usb/ehci_def.h
> ===================================================================
> --- usb-4.x.orig/include/linux/usb/ehci_def.h
> +++ usb-4.x/include/linux/usb/ehci_def.h
> @@ -180,11 +180,11 @@ struct ehci_regs {
>   * PORTSCx
>   */
>       /* HOSTPC: offset 0x84 */
> -     u32             hostpc[1];      /* HOSTPC extension */
> +     u32             hostpc[0];      /* HOSTPC extension */
>  #define HOSTPC_PHCD  (1<<22)         /* Phy clock disable */
>  #define HOSTPC_PSPD  (3<<25)         /* Port speed detection */
>  
> -     u32             reserved5[16];
> +     u32             reserved5[17];
>  
>       /* USBMODE_EX: offset 0xc8 */
>       u32             usbmode_ex;     /* USB Device mode extension */

--
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