hwmon: (lm85) Fix error paths in probe function
authorJean Delvare <khali@linux-fr.org>
Fri, 29 Apr 2011 14:33:36 +0000 (16:33 +0200)
committerJean Delvare <khali@endymion.delvare>
Fri, 29 Apr 2011 14:33:36 +0000 (16:33 +0200)
We must remove all files we created, even in error cases.

Fixes second part of kernel bug #34072:
https://bugzilla.kernel.org/show_bug.cgi?id=34072

Signed-off-by: Jean Delvare <khali@linux-fr.org>
Acked-by: Guenter Roeck <guenter.roeck@ericsson.com>
drivers/hwmon/lm85.c

index b5a6bc48597e82aa3ee336ba49ba1659c27110be..da72dc12068c11f85769f93aa57a6370ffa2ffd0 100644 (file)
@@ -1331,11 +1331,11 @@ static int lm85_probe(struct i2c_client *client,
        if (data->type != emc6d103s) {
                err = sysfs_create_group(&client->dev.kobj, &lm85_group_minctl);
                if (err)
-                       goto err_kfree;
+                       goto err_remove_files;
                err = sysfs_create_group(&client->dev.kobj,
                                         &lm85_group_temp_off);
                if (err)
-                       goto err_kfree;
+                       goto err_remove_files;
        }
 
        /* The ADT7463/68 have an optional VRM 10 mode where pin 21 is used