On Thu, 8 Dec 2016 11:22:29 -0800, Guenter Roeck wrote: > Module test reports: > > in0_min: Suspected overflow: [3320 vs. 0] > in0_max: Suspected overflow: [3320 vs. 0] > in4_min: Suspected overflow: [15938 vs. 0] > in4_max: Suspected overflow: [15938 vs. 0] > temp1_max: Suspected overflow: [127000 vs. 0] > temp1_max_hyst: Suspected overflow: [127000 vs. 0] > aout_output: Suspected overflow: [1250 vs. 0] > > Code analysis reveals that the overflows are caused by conversions > from unsigned long to long to int, combined with multiplications on > passed values. > > Signed-off-by: Guenter Roeck <linux@xxxxxxxxxxxx> > --- > v2: Improved and simplified clamping > > drivers/hwmon/adm9240.c | 9 ++++++--- > 1 file changed, 6 insertions(+), 3 deletions(-) > (...) Reviewed-by: Jean Delvare <jdelvare@xxxxxxx> -- Jean Delvare SUSE L3 Support -- To unsubscribe from this list: send the line "unsubscribe linux-hwmon" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html