Re: UBSAN splat in drivers/usb/host/ehci-hub.c:889:34

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

 



Am Wed, Jun 22, 2016 at 04:25:21PM -0400 schrieb Alan Stern:
> On Wed, 22 Jun 2016, Wilfried Klaebe wrote:
> 
> > Am Wed, Jun 22, 2016 at 07:56:49AM -0700 schrieb Greg KH:
> > > On Wed, Jun 22, 2016 at 12:11:08PM +0200, Wilfried Klaebe wrote:
> > > > Hi.
> > > > 
> > > > In linux-4.7-rc3, I activated CONFIG_UBSAN and started reporting results
> > > > by opening bugs in bugzilla.kernel.org. This was first about a UBSAN
> > > > splat in drivers/usb/host/ehci-hub.c:877:47 about which I wrote in
> > > > <https://bugzilla.kernel.org/show_bug.cgi?id=120361>. GregKH asked if
> > > > this was fixed in linux-next, but in linux-next-20160616 there was a
> > > > different splat (index 2 out of range instead of index -1 out of range)
> > > > in drivers/usb/host/ehci-hub.c:889:34. I got asked to report here.
> > > > That last splat persists until at least linux-next-20160622.
> > > > 
> > > > Is there anything else you need?
> > > 
> > > Is this still in 4.7-rc4?  All of the reported ubsan fixes for USB
> > > should now be in there.
> > 
> > It is.
> > 
> > > If not, can you provide the full "splat" here?
> > 
> > dmesg attached, including UBSAN message and following further ehci messages.

> Please test this patch.

The UBSAN complaint about ehci-hub.c is gone with this patch.

Tested-By: Wilfried Klaebe <linux-kernel@xxxxxxxxxxxxxxxxxxxxxxxxxx>

> By coincidence, it turns out that I'm the person responsible for the
> other UBSAN complaint in your log.  Please see if the following patch
> fixes it.

That patch makes the scsi_devinfo.c UBSAN compaint go away for me.

Tested-By: Wilfried Klaebe <linux-kernel@xxxxxxxxxxxxxxxxxxxxxxxxxx>

Regards,
Wilfried
--
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