- Add two missing functions to the LibUSB v0.1 API.

- Clamp the string length to 255 bytes when getting
  the interface description.
- Clamp data request length to 65535 bytes when doing
  control requests.

MFC after:	3 days
This commit is contained in:
Hans Petter Selasky 2011-07-16 08:29:12 +00:00
parent dcded131e1
commit 4eb5923d1a
6 changed files with 87 additions and 0 deletions

View File

@ -542,6 +542,8 @@ The library is also compliant with LibUSB version 0.1.12.
.Fn usb_device .Fn usb_device
.Fn usb_get_busses .Fn usb_get_busses
.Fn usb_check_connected .Fn usb_check_connected
.Fn usb_get_driver_np
.Fn usb_detach_kernel_driver_np
. .
.Sh SEE ALSO .Sh SEE ALSO
.Xr libusb20 3 , .Xr libusb20 3 ,

View File

@ -203,6 +203,12 @@ usb_get_string(usb_dev_handle * dev, int strindex,
{ {
int err; int err;
if (dev == NULL)
return (-1);
if (buflen > 65535)
buflen = 65535;
err = libusb20_dev_req_string_sync((void *)dev, err = libusb20_dev_req_string_sync((void *)dev,
strindex, langid, buf, buflen); strindex, langid, buf, buflen);
@ -218,6 +224,12 @@ usb_get_string_simple(usb_dev_handle * dev, int strindex,
{ {
int err; int err;
if (dev == NULL)
return (-1);
if (buflen > 65535)
buflen = 65535;
err = libusb20_dev_req_string_simple_sync((void *)dev, err = libusb20_dev_req_string_simple_sync((void *)dev,
strindex, buf, buflen); strindex, buf, buflen);
@ -233,6 +245,12 @@ usb_get_descriptor_by_endpoint(usb_dev_handle * udev, int ep, uint8_t type,
{ {
memset(buf, 0, size); memset(buf, 0, size);
if (udev == NULL)
return (-1);
if (size > 65535)
size = 65535;
return (usb_control_msg(udev, ep | USB_ENDPOINT_IN, return (usb_control_msg(udev, ep | USB_ENDPOINT_IN,
USB_REQ_GET_DESCRIPTOR, (type << 8) + ep_index, 0, USB_REQ_GET_DESCRIPTOR, (type << 8) + ep_index, 0,
buf, size, 1000)); buf, size, 1000));
@ -244,6 +262,12 @@ usb_get_descriptor(usb_dev_handle * udev, uint8_t type, uint8_t desc_index,
{ {
memset(buf, 0, size); memset(buf, 0, size);
if (udev == NULL)
return (-1);
if (size > 65535)
size = 65535;
return (usb_control_msg(udev, USB_ENDPOINT_IN, USB_REQ_GET_DESCRIPTOR, return (usb_control_msg(udev, USB_ENDPOINT_IN, USB_REQ_GET_DESCRIPTOR,
(type << 8) + desc_index, 0, buf, size, 1000)); (type << 8) + desc_index, 0, buf, size, 1000));
} }
@ -943,3 +967,49 @@ usb_get_busses(void)
{ {
return (usb_busses); return (usb_busses);
} }
int
usb_get_driver_np(usb_dev_handle * dev, int interface, char *name, int namelen)
{
struct libusb20_device *pdev;
char *ptr;
int err;
pdev = (void *)dev;
if (pdev == NULL)
return (-1);
if (namelen < 1)
return (-1);
if (namelen > 255)
namelen = 255;
err = libusb20_dev_get_iface_desc(pdev, interface, name, namelen);
if (err != 0)
return (-1);
/* we only want the driver name */
ptr = strstr(name, ":");
if (ptr != NULL)
*ptr = 0;
return (0);
}
int
usb_detach_kernel_driver_np(usb_dev_handle * dev, int interface)
{
struct libusb20_device *pdev;
int err;
pdev = (void *)dev;
if (pdev == NULL)
return (-1);
err = libusb20_dev_detach_kernel_driver(pdev, interface);
if (err != 0)
return (-1);
return (0);
}

View File

@ -719,6 +719,8 @@ libusb_get_driver(struct libusb20_device *pdev, int interface,
return (LIBUSB_ERROR_INVALID_PARAM); return (LIBUSB_ERROR_INVALID_PARAM);
if (namelen < 1) if (namelen < 1)
return (LIBUSB_ERROR_INVALID_PARAM); return (LIBUSB_ERROR_INVALID_PARAM);
if (namelen > 255)
namelen = 255;
err = libusb20_dev_get_iface_desc( err = libusb20_dev_get_iface_desc(
pdev, interface, name, namelen); pdev, interface, name, namelen);

View File

@ -300,6 +300,9 @@ libusb_get_string_descriptor_ascii(libusb_device_handle *pdev,
if (pdev == NULL || data == NULL || length < 1) if (pdev == NULL || data == NULL || length < 1)
return (LIBUSB20_ERROR_INVALID_PARAM); return (LIBUSB20_ERROR_INVALID_PARAM);
if (length > 65535)
length = 65535;
/* put some default data into the destination buffer */ /* put some default data into the destination buffer */
data[0] = 0; data[0] = 0;
@ -314,6 +317,12 @@ int
libusb_get_descriptor(libusb_device_handle * devh, uint8_t desc_type, libusb_get_descriptor(libusb_device_handle * devh, uint8_t desc_type,
uint8_t desc_index, uint8_t *data, int length) uint8_t desc_index, uint8_t *data, int length)
{ {
if (devh == NULL || data == NULL || length < 1)
return (LIBUSB20_ERROR_INVALID_PARAM);
if (length > 65535)
length = 65535;
return (libusb_control_transfer(devh, LIBUSB_ENDPOINT_IN, return (libusb_control_transfer(devh, LIBUSB_ENDPOINT_IN,
LIBUSB_REQUEST_GET_DESCRIPTOR, (desc_type << 8) | desc_index, 0, data, LIBUSB_REQUEST_GET_DESCRIPTOR, (desc_type << 8) | desc_index, 0, data,
length, 1000)); length, 1000));

View File

@ -1081,6 +1081,8 @@ libusb20_dev_get_iface_desc(struct libusb20_device *pdev,
if ((buf == NULL) || (len == 0)) if ((buf == NULL) || (len == 0))
return (LIBUSB20_ERROR_INVALID_PARAM); return (LIBUSB20_ERROR_INVALID_PARAM);
buf[0] = 0; /* set default string value */
return (pdev->beMethods->dev_get_iface_desc( return (pdev->beMethods->dev_get_iface_desc(
pdev, iface_index, buf, len)); pdev, iface_index, buf, len));
} }

View File

@ -299,6 +299,8 @@ int usb_find_busses(void);
int usb_find_devices(void); int usb_find_devices(void);
struct usb_device *usb_device(usb_dev_handle * dev); struct usb_device *usb_device(usb_dev_handle * dev);
struct usb_bus *usb_get_busses(void); struct usb_bus *usb_get_busses(void);
int usb_get_driver_np(usb_dev_handle * dev, int interface, char *name, int namelen);
int usb_detach_kernel_driver_np(usb_dev_handle * dev, int interface);
#if 0 #if 0
{ /* style */ { /* style */