hid: Don't interpret the report ID from incoming feature reports.

The USB HID GET_REPORT(Feature) request includes the report ID that the
host is requesting in the SETUP packet. When the report ID is not zero
the device will send this back to the host as the first byte of the
report. Unlike input and output reports we want to preserve this part of
the report because the device may in fact by sending a value other than
the report ID as the first byte of the response.

To achieve this, on Windows this patch stops stripping the report ID
from the buffer received from the OS. On OS X this was already the
behavior. On Linux this was the behavior if the report ID was non-zero,
otherwise it would artificially prepend a zero to the buffer which must
be removed.

BUG=420112

Review URL: https://codereview.chromium.org/623963003

Cr-Commit-Position: refs/heads/master@{#298135}

1 files changed, 9 insertions, 1 deletions

diff --git a/device/hid/hid_connection_linux.cc b/device/hid/hid_connection_linux.cc
index ac2e554..936f016 100644
--- a/device/hid/hid_connection_linux.cc
+++ b/device/hid/hid_connection_linux.cc
@@ -110,7 +110,7 @@ void HidConnectionLinux::PlatformGetFeatureReport(
   // and is overwritten by the feature report.
   DCHECK_GT(device_info().max_feature_report_size, 0);
   scoped_refptr<net::IOBufferWithSize> buffer(
-      new net::IOBufferWithSize(device_info().max_feature_report_size));
+      new net::IOBufferWithSize(device_info().max_feature_report_size + 1));
   buffer->data()[0] = report_id;
 
   int result = ioctl(device_file_.GetPlatformFile(),
@@ -119,6 +119,14 @@ void HidConnectionLinux::PlatformGetFeatureReport(
   if (result < 0) {
     VPLOG(1) << "Failed to get feature report";
     callback.Run(false, NULL, 0);
+  } else if (result == 0) {
+    VLOG(1) << "Get feature result too short.";
+    callback.Run(false, NULL, 0);
+  } else if (report_id == 0) {
+    // Linux adds a 0 to the beginning of the data received from the device.
+    scoped_refptr<net::IOBuffer> copied_buffer(new net::IOBuffer(result - 1));
+    memcpy(copied_buffer->data(), buffer->data() + 1, result - 1);
+    callback.Run(true, copied_buffer, result - 1);
   } else {
     callback.Run(true, buffer, result);
   }