]> www.infradead.org Git - users/hch/xfs.git/commit
USB: serial: mos7840: fix crash on resume
authorDmitry Smirnov <d.smirnov@inbox.lv>
Fri, 14 Jun 2024 22:45:56 +0000 (01:45 +0300)
committerJohan Hovold <johan@kernel.org>
Tue, 9 Jul 2024 16:10:04 +0000 (18:10 +0200)
commitc15a688e49987385baa8804bf65d570e362f8576
tree47408be51a6981e10570cb90e683bfb5f0ea9ff8
parentae420771551bd9f04347c59744dd062332bdec3e
USB: serial: mos7840: fix crash on resume

Since commit c49cfa917025 ("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.

Signed-off-by: Dmitry Smirnov <d.smirnov@inbox.lv>
[ johan: analyse crash and rewrite commit message; set busy flag on
         resume; drop bulk-in check; drop unnecessary usb_kill_urb() ]
Fixes: d83b405383c9 ("USB: serial: add support for multiple read urbs")
Cc: stable@vger.kernel.org # 3.3
Signed-off-by: Johan Hovold <johan@kernel.org>
drivers/usb/serial/mos7840.c