ACK: [SRU][F][PATCH 0/1] CVE-2024-42244

[Thibault Ferrante]

Acked-by: Thibault Ferrante <thibault.ferrante at canonical.com>


On 16-09-2024 22:40, Bethany Jamison wrote:
> [Impact]
> 
> USB: serial: mos7840: fix crash on resume
> 
> Since commit c49cfa9 ("USB: serial: use generic method if no
> alternative is provided in usb serial layer"), USB serial core calls the
> generic resume implementation when the driver has not provided one.
> 
> This can trigger a crash on resume with mos7840 since support for
> multiple read URBs was added back in 2011. Specifically, both port read
> URBs are now submitted on resume for open ports, but the context pointer
> of the second URB is left set to the core rather than mos7840 port
> structure.
> 
> Fix this by implementing dedicated suspend and resume functions for
> mos7840.
> 
> Tested with Delock 87414 USB 2.0 to 4x serial adapter.
> 
> [Fix]
> 
> Noble:	pending (6.8.0-46.46)
> Jammy:	released
> Focal:	Backported from linux-5.10.y - I adjusted the code to ignore a
> 	context conflict, Focal has '.read_int_callback' line which isn't
> 	upstream
> Bionic:	fix sent to esm ML
> Xenial:	fix needed
> Trusty:	won't fix
> 
> [Test Case]
> 
> Compile tested.
> 
> [Where problems could occur]
> 
> This fix affects those who use the Moschip 7840/7820 USB Serial Driver,
> an issue with this fix would be visible to the user via a system crash.
> 
> Dmitry Smirnov (1):
>    USB: serial: mos7840: fix crash on resume
> 
>   drivers/usb/serial/mos7840.c | 45 ++++++++++++++++++++++++++++++++++++
>   1 file changed, 45 insertions(+)
> 


-- 
--
Thibault