Missing curly braces cause an if statement to be evaluated when it
shouldn't. It happens to be harmless, but that's still worth fixing.
Thanks to Riku Voipio for reporting.
Signed-off-by: Jean Delvare <khali at linux-fr.org>
---
drivers/hwmon/w83781d.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
--- linux-2.6.24-rc1.orig/drivers/hwmon/w83781d.c 2007-10-25 12:58:24.000000000 +0200
+++ linux-2.6.24-rc1/drivers/hwmon/w83781d.c 2007-10-25 13:06:00.000000000 +0200
@@ -1121,12 +1121,13 @@ w83781d_create_files(struct device *dev,
&sensor_dev_attr_temp3_beep.dev_attr)))
return err;
- if (kind != w83781d)
+ if (kind != w83781d) {
err = sysfs_chmod_file(&dev->kobj,
&sensor_dev_attr_temp3_alarm.dev_attr.attr,
S_IRUGO | S_IWUSR);
if (err)
return err;
+ }
}
if (kind != w83781d && kind != as99127f) {
--
Jean Delvare
