diff options
author | Gustavo A. R. Silva <garsilva@embeddedor.com> | 2017-02-16 17:27:36 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-03-16 04:58:44 -0400 |
commit | 78f74f75bd80b8d3cc731da91d2065b172c67817 (patch) | |
tree | b7b6bf56be801634be7ca800111d355e0028b0cb | |
parent | 4495c08e84729385774601b5146d51d9e5849f81 (diff) |
usb: atm: remove unnecessary code
'index' is an unsigned variable, and less-than-zero comparison of an unsigned
variable is never true.
Addresses-Coverity-ID: 115396
Signed-off-by: Gustavo A. R. Silva <garsilva@embeddedor.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | drivers/usb/atm/cxacru.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/usb/atm/cxacru.c b/drivers/usb/atm/cxacru.c index f9fe86b6f7b5..d65a64c29b85 100644 --- a/drivers/usb/atm/cxacru.c +++ b/drivers/usb/atm/cxacru.c | |||
@@ -474,7 +474,7 @@ static ssize_t cxacru_sysfs_store_adsl_config(struct device *dev, | |||
474 | ret = sscanf(buf + pos, "%x=%x%n", &index, &value, &tmp); | 474 | ret = sscanf(buf + pos, "%x=%x%n", &index, &value, &tmp); |
475 | if (ret < 2) | 475 | if (ret < 2) |
476 | return -EINVAL; | 476 | return -EINVAL; |
477 | if (index < 0 || index > 0x7f) | 477 | if (index > 0x7f) |
478 | return -EINVAL; | 478 | return -EINVAL; |
479 | if (tmp < 0 || tmp > len - pos) | 479 | if (tmp < 0 || tmp > len - pos) |
480 | return -EINVAL; | 480 | return -EINVAL; |