]> git.itanic.dy.fi Git - linux-stable/commit
usb: xhci: Handle error condition in xhci_stop_device()
authorMayank Rana <mrana@codeaurora.org>
Fri, 6 Oct 2017 14:45:30 +0000 (17:45 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 17 Oct 2017 08:38:13 +0000 (10:38 +0200)
commitb3207c65dfafae27e7c492cb9188c0dc0eeaf3fd
treee2e01323c0bd94fdb7df61652cd455611e7c971a
parent810a624bd1b64b13ddcc2eb5c1880526a750a870
usb: xhci: Handle error condition in xhci_stop_device()

xhci_stop_device() calls xhci_queue_stop_endpoint() multiple times
without checking the return value. xhci_queue_stop_endpoint() can
return error if the HC is already halted or unable to queue commands.
This can cause a deadlock condition as xhci_stop_device() would
end up waiting indefinitely for a completion for the command that
didn't get queued. Fix this by checking the return value and bailing
out of xhci_stop_device() in case of error. This patch happens to fix
potential memory leaks of the allocated command structures as well.

Fixes: c311e391a7ef ("xhci: rework command timeout and cancellation,")
Cc: <stable@vger.kernel.org>
Signed-off-by: Mayank Rana <mrana@codeaurora.org>
Signed-off-by: Jack Pham <jackp@codeaurora.org>
Signed-off-by: Mathias Nyman <mathias.nyman@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/host/xhci-hub.c