diff options
author | Axel Lin <axel.lin@ingics.com> | 2014-08-05 20:24:00 -0400 |
---|---|---|
committer | Guenter Roeck <linux@roeck-us.net> | 2014-08-05 22:44:42 -0400 |
commit | fa642d9d6e502fbdbd1122a6a247c8ad17063a29 (patch) | |
tree | bd56997463057d2127fdd6777d70ecf65e668c38 | |
parent | db59ac434230c2d6c051bb314f320ad977e6841d (diff) |
hwmon: (lm87) Fix vrm write operation
vrm is an u8, so the written value needs to be limited to [0, 255].
Signed-off-by: Axel Lin <axel.lin@ingics.com>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
-rw-r--r-- | drivers/hwmon/lm87.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/hwmon/lm87.c b/drivers/hwmon/lm87.c index ba1d83d48056..a5e295826aea 100644 --- a/drivers/hwmon/lm87.c +++ b/drivers/hwmon/lm87.c | |||
@@ -617,6 +617,10 @@ static ssize_t set_vrm(struct device *dev, struct device_attribute *attr, | |||
617 | err = kstrtoul(buf, 10, &val); | 617 | err = kstrtoul(buf, 10, &val); |
618 | if (err) | 618 | if (err) |
619 | return err; | 619 | return err; |
620 | |||
621 | if (val > 255) | ||
622 | return -EINVAL; | ||
623 | |||
620 | data->vrm = val; | 624 | data->vrm = val; |
621 | return count; | 625 | return count; |
622 | } | 626 | } |