W dniu 18.05.2015 o 20:24, Dan Carpenter pisze:
Hello Andrzej Pietrasiewicz,
The patch d6d22922d907: "usb: gadget: rndis: remove the limit of
available rndis connections" from Feb 6, 2015, leads to the following
static checker warning:
drivers/usb/gadget/function/rndis.c:881 rndis_register()
warn: impossible condition '(i < 0) => (0-255 < 0)'
drivers/usb/gadget/function/rndis.c
872 struct rndis_params *rndis_register(void (*resp_avail)(void *v), void *v)
873 {
874 struct rndis_params *params;
875 u8 i;
876
877 if (!resp_avail)
878 return ERR_PTR(-EINVAL);
879
880 i = rndis_get_nr();
881 if (i < 0) {
^^^^^
i is unsigned. I think it needs to be int to hold the high values as
well.
There is already a patch series:
http://www.spinics.net/lists/linux-usb/msg125003.html
AP
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
