diff options
| author |   Axel Lin <axel.lin@ingics.com> | 2014-08-06 08:27:11 +0800 |
|---|---|---|
| committer |   Guenter Roeck <linux@roeck-us.net> | 2014-08-05 19:44:42 -0700 |
| commit | 970255b75d0b54e457091182a7ae3a188ac0c346 (patch) | |
| tree | b8a1309772eca7b66a1e729c87f3082557ffc085 /drivers/hwmon/w83627hf.c | |
| parent | hwmon: (vt1211) Fix vrm write operation (diff) | |
| download | linux-dev-970255b75d0b54e457091182a7ae3a188ac0c346.tar.xz linux-dev-970255b75d0b54e457091182a7ae3a188ac0c346.zip | |
hwmon: (w83627hf) 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>
Diffstat (limited to 'drivers/hwmon/w83627hf.c')
| -rw-r--r-- | drivers/hwmon/w83627hf.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/hwmon/w83627hf.c b/drivers/hwmon/w83627hf.c index c1726be3654c..2f55973a8c4c 100644 --- a/drivers/hwmon/w83627hf.c +++ b/drivers/hwmon/w83627hf.c @@ -820,6 +820,9 @@ store_vrm_reg(struct device *dev, struct device_attribute *attr, const char *buf err = kstrtoul(buf, 10, &val); if (err) return err; + + if (val > 255) + return -EINVAL; data->vrm = val; return count; |