On Mon, Mar 27, 2017 at 02:17:48PM +0200, Peter Rosin wrote: > Comparing a size_t with less than zero is always false as size_t > is unsigned. So, change the type of the variable to ssize_t and > replicate the size check from mux_configure_channel() into > mux_write_ext_info() thus ensuring that the size will fit in the > ssize_t variable. > > Detected by CoverityScan, CID#1415278 ("Unsigned compared against 0") > > Fixes: 1da8e16d2812 ("iio: multiplexer: new iio category and iio-mux driver") You should fold this one and the next patch into the patches that they fix up instead fixing stuff that's not yet merged incrementally like you do here (specifically, the SHA id above will have no matching commit). Johan -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html