ub: Bind to first endpoint, not to last

The usb-storage switched to binding to first endpoint recently. Apparently,
there are devices out there with extra endpoints. It is perfectly legal.

Signed-off-by: Pete Zaitcev <zaitcev@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

1 files changed, 7 insertions, 4 deletions

diff --git a/drivers/block/ub.c b/drivers/block/ub.c
index 2098eff91e14..746a118a9b52 100644
--- a/drivers/block/ub.c
+++ b/drivers/block/ub.c
@@ -2132,10 +2132,13 @@ static int ub_get_pipes(struct ub_dev *sc, struct usb_device *dev,
                 if ((ep->bmAttributes & USB_ENDPOINT_XFERTYPE_MASK)
                                 == USB_ENDPOINT_XFER_BULK) {
                         /* BULK in or out? */
-                        if (ep->bEndpointAddress & USB_DIR_IN)
-                                ep_in = ep;
-                        else
-                                ep_out = ep;
+                        if (ep->bEndpointAddress & USB_DIR_IN) {
+                                if (ep_in == NULL)
+                                        ep_in = ep;
+                        } else {
+                                if (ep_out == NULL)
+                                        ep_out = ep;
+                        }
                 }
         }