On Thu, Sep 17, 2020 at 09:56:21PM -0400, Sasha Levin wrote: > From: Johan Hovold <jo...@kernel.org> > > [ Upstream commit 960fbd1ca584a5b4cd818255769769d42bfc6dbe ] > > The driver would return success and leave the port structures > half-initialised if any of the register accesses during probe fails. > > This would specifically leave the port control urb unallocated, > something which could trigger a NULL pointer dereference on interrupt > events. > > Fortunately the interrupt implementation is completely broken and has > never even been enabled... > > Note that the zero-length-enable register write used to set the zle-flag > for all ports is moved to attach. > > Reviewed-by: Greg Kroah-Hartman <gre...@linuxfoundation.org> > Signed-off-by: Johan Hovold <jo...@kernel.org> > Signed-off-by: Sasha Levin <sas...@kernel.org>
Please drop this from all stable queues. As the commit message and missing stable-cc tag suggests, it's not needed. Sasha, please stop sending AUTOSEL patches for usb-serial. I think this the fourth time I ask you now. Johan
