On Sun, 4 Dec 2016 20:55:36 -0800, Guenter Roeck wrote: > Writes into temperature limit attributes can overflow due to unbound > values passed to DIV_ROUND_CLOSEST(). > > Cc: Steve Glendinning <steve.glendinn...@shawell.net> > Signed-off-by: Guenter Roeck <li...@roeck-us.net> > --- > drivers/hwmon/emc2103.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/hwmon/emc2103.c b/drivers/hwmon/emc2103.c > index 24e395c5907d..4b870ee9b0d3 100644 > --- a/drivers/hwmon/emc2103.c > +++ b/drivers/hwmon/emc2103.c > @@ -251,7 +251,7 @@ static ssize_t set_temp_min(struct device *dev, struct > device_attribute *da, > if (result < 0) > return result; > > - val = clamp_val(DIV_ROUND_CLOSEST(val, 1000), -63, 127); > + val = DIV_ROUND_CLOSEST(clamp_val(val, -63000, 127000), 1000); > > mutex_lock(&data->update_lock); > data->temp_min[nr] = val; > @@ -273,7 +273,7 @@ static ssize_t set_temp_max(struct device *dev, struct > device_attribute *da, > if (result < 0) > return result; > > - val = clamp_val(DIV_ROUND_CLOSEST(val, 1000), -63, 127); > + val = DIV_ROUND_CLOSEST(clamp_val(val, -63000, 127000), 1000); > > mutex_lock(&data->update_lock); > data->temp_max[nr] = val;
Reviewed-by: Jean Delvare <jdelv...@suse.de> -- Jean Delvare SUSE L3 Support -- To unsubscribe from this list: send the line "unsubscribe linux-hwmon" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
