On Thu, Mar 27, 2014 at 06:45:07PM +0300, Dan Carpenter wrote:
> > drivers/net/wireless/rsi/rsi_91x_usb.c:289
> > rsi_usb_write_register_multiple() warn: '__min2' 4096 can't fit into 255
> > 'transfer'
The code for this one is:
transfer = min_t(int, count, 4096);
It's complaining because "tranfers" is u8, but the cast to int always
makes me worry negative values. In this case, count is not user
controlled so it's safe. But that's a common security problem you
should be aware of. It's better to cast it to an unsigned value.
regards,
dan carpenter
_______________________________________________
kbuild mailing list
kbuild@lists.01.org
https://lists.01.org/mailman/listinfo/kbuild
