summaryrefslogtreecommitdiffstats
path: root/drivers/usb
diff options
context:
space:
mode:
authorFelipe Balbi <felipe.balbi@linux.intel.com>2017-11-09 12:59:58 +0200
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2017-11-09 13:02:23 +0100
commitd656fa32b13424701bb85f6b5183ea869b363799 (patch)
tree76cbfb42913ac32bf6d45c1d777c3e4d5b5f41c8 /drivers/usb
parent0c3b34a569f83b281c4c0ef4542cf59d7b186a88 (diff)
downloadop-kernel-dev-d656fa32b13424701bb85f6b5183ea869b363799.zip
op-kernel-dev-d656fa32b13424701bb85f6b5183ea869b363799.tar.gz
usb: core: message: remember to reset 'ret' to 0 when necessary
usb_control_msg() will return the amount of bytes transferred, if that amount matches what we wanted to transfer, we need to reset 'ret' to 0 from usb_get_status(). Fixes: 2e43f0fe379c ("usb: core: add a 'type' parameter to usb_get_status()") Reported-by: Tony Lindgren <tony@atomide.com> Signed-off-by: Felipe Balbi <felipe.balbi@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/usb')
-rw-r--r--drivers/usb/core/message.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/usb/core/message.c b/drivers/usb/core/message.c
index ebaea51..77001bc 100644
--- a/drivers/usb/core/message.c
+++ b/drivers/usb/core/message.c
@@ -975,6 +975,7 @@ int usb_get_status(struct usb_device *dev, int recip, int type, int target,
}
*(u32 *) data = le32_to_cpu(*(__le32 *) status);
+ ret = 0;
break;
case 2:
if (type != USB_STATUS_TYPE_STANDARD) {
@@ -983,6 +984,7 @@ int usb_get_status(struct usb_device *dev, int recip, int type, int target,
}
*(u16 *) data = le16_to_cpu(*(__le16 *) status);
+ ret = 0;
break;
default:
ret = -EIO;
OpenPOWER on IntegriCloud