staging: ozwpan: Return correct actual_length to userland
authorRupesh Gujare <rgujare@ozmodevices.com>
Mon, 23 Jul 2012 17:49:45 +0000 (18:49 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 14 Aug 2012 02:17:17 +0000 (19:17 -0700)
This fixes issue where wrong retrun value was received
by userland application after writing data to raw hid device.

Signed-off-by: Rupesh Gujare <rgujare@ozmodevices.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/ozwpan/ozhcd.c

index 4ac1f27..76821cb 100644 (file)
@@ -933,13 +933,14 @@ void oz_hcd_control_cnf(void *hport, u8 req_id, u8 rcode, u8 *data,
        } else {
                int copy_len;
                oz_trace("VENDOR-CLASS - cnf\n");
-               if (data_len <= urb->transfer_buffer_length)
-                       copy_len = data_len;
-               else
-                       copy_len = urb->transfer_buffer_length;
-               if (copy_len)
+               if (data_len) {
+                       if (data_len <= urb->transfer_buffer_length)
+                               copy_len = data_len;
+                       else
+                               copy_len = urb->transfer_buffer_length;
                        memcpy(urb->transfer_buffer, data, copy_len);
-               urb->actual_length = copy_len;
+                       urb->actual_length = copy_len;
+               }
                oz_complete_urb(hcd, urb, 0, 0);
        }
 }
@@ -1517,6 +1518,7 @@ static void oz_process_ep0_urb(struct oz_hcd *ozhcd, struct urb *urb,
                int data_len = 0;
                if ((setup->bRequestType & USB_DIR_IN) == 0)
                        data_len = wlength;
+               urb->actual_length = data_len;
                if (oz_usb_control_req(port->hpd, req_id, setup,
                                urb->transfer_buffer, data_len)) {
                        rc = -ENOMEM;