diff options
author | roel kluin <roel.kluin@gmail.com> | 2008-10-21 00:36:44 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2008-10-22 13:05:28 -0400 |
commit | 8aafdf6a1528b0672a0527be473d9be5a12289d3 (patch) | |
tree | a2fcd648b5c17b554f8197f0f5d4fd6fe172c954 | |
parent | 1460e5e44cc5ecad7704f63b10dcb3a59d0e008b (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.c | 3 |
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)) |