Hi

This patch fixes a bug (discovered by Hans De Goede) in wince_get_active_config_descriptor(), which is erroneously setting the host_endian parameter to 1 when it should be set to 0.

Many thanks

Simon
>From 2defb08b87ac721b331d4393c846f1939f21f9c5 Mon Sep 17 00:00:00 2001
From: Simon Haggett <simon.hagg...@realvnc.com>
Date: Mon, 20 May 2013 12:24:00 +0100
Subject: [PATCH] WinCE: wince_get_active_config_descriptor() sets host_endian
 incorrectly

wince_get_active_config_descriptor() retrieves configuration descriptors
as raw bytes, in bus-endian order. Therefore, host_endian should be set
to 0 (as in wince_get_config_descriptor()).
---
 libusb/os/wince_usb.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libusb/os/wince_usb.c b/libusb/os/wince_usb.c
index 2e6eedd..d526778 100644
--- a/libusb/os/wince_usb.c
+++ b/libusb/os/wince_usb.c
@@ -435,7 +435,7 @@ static int wince_get_active_config_descriptor(
 {
 	struct wince_device_priv *priv = _device_priv(device);
 	DWORD actualSize = len;
-	*host_endian = 1;
+	*host_endian = 0;
 	if (!UkwGetConfigDescriptor(priv->dev, UKW_ACTIVE_CONFIGURATION, buffer, len, &actualSize)) {
 		return translate_driver_error(GetLastError());
 	}
-- 
1.8.1.msysgit.1

------------------------------------------------------------------------------
AlienVault Unified Security Management (USM) platform delivers complete
security visibility with the essential security capabilities. Easily and
efficiently configure, manage, and operate all of your security controls
from a single console and one unified framework. Download a free trial.
http://p.sf.net/sfu/alienvault_d2d
_______________________________________________
libusbx-devel mailing list
libusbx-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/libusbx-devel

Reply via email to