aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorroel kluin <roel.kluin@gmail.com>2008-10-21 00:36:44 -0400
committerGreg Kroah-Hartman <gregkh@suse.de>2008-10-22 13:05:28 -0400
commit8aafdf6a1528b0672a0527be473d9be5a12289d3 (patch)
treea2fcd648b5c17b554f8197f0f5d4fd6fe172c954
parent1460e5e44cc5ecad7704f63b10dcb3a59d0e008b (diff)
USB: usbtest.c: length, sglen and vary are unsigned, so cannot be negative
length, sglen and vary are unsigned, so cannot be negative see vi drivers/usb/misc/usbtest.c +18 struct usbtest_param { ... unsigned iterations; unsigned length; unsigned vary; unsigned sglen; ... }; Signed-off-by: Roel Kluin <roel.kluin@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-rw-r--r--drivers/usb/misc/usbtest.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/usb/misc/usbtest.c b/drivers/usb/misc/usbtest.c
index b358c4e1cf21..444c69c447be 100644
--- a/drivers/usb/misc/usbtest.c
+++ b/drivers/usb/misc/usbtest.c
@@ -1561,8 +1561,7 @@ usbtest_ioctl (struct usb_interface *intf, unsigned int code, void *buf)
1561 if (code != USBTEST_REQUEST) 1561 if (code != USBTEST_REQUEST)
1562 return -EOPNOTSUPP; 1562 return -EOPNOTSUPP;
1563 1563
1564 if (param->iterations <= 0 || param->length < 0 1564 if (param->iterations <= 0)
1565 || param->sglen < 0 || param->vary < 0)
1566 return -EINVAL; 1565 return -EINVAL;
1567 1566
1568 if (mutex_lock_interruptible(&dev->lock)) 1567 if (mutex_lock_interruptible(&dev->lock))