[lm-sensors] [PATCH] hwmon: (w83781d) Add missing curly braces

Jean Delvare khali at linux-fr.org
Thu Oct 25 13:11:01 CEST 2007


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




More information about the lm-sensors mailing list