summaryrefslogtreecommitdiffstats
path: root/drivers/usb
diff options
context:
space:
mode:
authorSarah Sharp <sarah.a.sharp@linux.intel.com>2012-10-16 13:26:22 -0700
committerSarah Sharp <sarah.a.sharp@linux.intel.com>2012-10-25 13:13:48 -0700
commitb80313422a1551cc1d4f3e2d532fe633154ae5dd (patch)
tree1b5ec47e20e17eeeaa11e52831e1c7e047ae70e4 /drivers/usb
parent43a09f7fb01fa1e091416a2aa49b6c666458c1ee (diff)
downloadop-kernel-dev-b80313422a1551cc1d4f3e2d532fe633154ae5dd.zip
op-kernel-dev-b80313422a1551cc1d4f3e2d532fe633154ae5dd.tar.gz
xhci: Fix missing break in xhci_evaluate_context_result.
Coverity complains that xhci_evaluate_context_result() is missing a break statement after the COMP_EBADSLT switch case. It's not a big deal, since we wanted to return the same error code as the case statement below it does. The end result would be one that a Slot Disabled error completion code would also print the warning message associated with a Context State error code. No other bad behavior would result. It's not worth backporting to stable kernels, since it only fixes an issue with too much debugging. Signed-off-by: Sarah Sharp <sarah.a.sharp@linux.intel.com>
Diffstat (limited to 'drivers/usb')
-rw-r--r--drivers/usb/host/xhci.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/usb/host/xhci.c b/drivers/usb/host/xhci.c
index 9ec9396..ffe2e2e 100644
--- a/drivers/usb/host/xhci.c
+++ b/drivers/usb/host/xhci.c
@@ -1817,6 +1817,8 @@ static int xhci_evaluate_context_result(struct xhci_hcd *xhci,
case COMP_EBADSLT:
dev_warn(&udev->dev, "WARN: slot not enabled for"
"evaluate context command.\n");
+ ret = -EINVAL;
+ break;
case COMP_CTX_STATE:
dev_warn(&udev->dev, "WARN: invalid context state for "
"evaluate context command.\n");
OpenPOWER on IntegriCloud