]> git.baikalelectronics.ru Git - kernel.git/commit
usb: dwc3: qcom: fix use-after-free on runtime-PM wakeup
authorJohan Hovold <johan+linaro@kernel.org>
Thu, 4 Aug 2022 15:09:56 +0000 (17:09 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 15 Sep 2022 10:04:51 +0000 (12:04 +0200)
commit9b73f03c5f30e27ca05d9ff5bcbea962a555b7c4
tree614a5669df0107ed24ad7b7f51d9c8f25523af6e
parentc7cdb7b7fa110e6eaded12e3beeac386ba0ff5e2
usb: dwc3: qcom: fix use-after-free on runtime-PM wakeup

[ Upstream commit 6a241819ad566ac21a5d76496f0c6123ed02e487 ]

The Qualcomm dwc3 runtime-PM implementation checks the xhci
platform-device pointer in the wakeup-interrupt handler to determine
whether the controller is in host mode and if so triggers a resume.

After a role switch in OTG mode the xhci platform-device would have been
freed and the next wakeup from runtime suspend would access the freed
memory.

Note that role switching is executed from a freezable workqueue, which
guarantees that the pointer is stable during suspend.

Also note that runtime PM has been broken since commit 5b1fe5eb5e5c
("usb: dwc3: qcom: Honor wakeup enabled/disabled state"), which
incidentally also prevents this issue from being triggered.

Fixes: 9f6dbe320417 ("usb: dwc3: Add Qualcomm DWC3 glue driver")
Cc: stable@vger.kernel.org # 4.18
Reviewed-by: Matthias Kaehlcke <mka@chromium.org>
Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Signed-off-by: Johan Hovold <johan+linaro@kernel.org>
Link: https://lore.kernel.org/r/20220804151001.23612-5-johan+linaro@kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/usb/dwc3/dwc3-qcom.c
drivers/usb/dwc3/host.c