From: Dan Carpenter <dan.carpenter@oracle.com>
Date: Wed, 15 Aug 2012 07:49:43 +0000 (+0300)
Subject: hwmon: (adt7410) handle errors from adt7410_update_device()
X-Git-Tag: next-20120917~87^2~15
X-Git-Url: https://git.karo-electronics.de/?a=commitdiff_plain;h=46f7691fdd8df291547ea5a4279ccb5d43a79d47;p=karo-tx-linux.git

hwmon: (adt7410) handle errors from adt7410_update_device()

Smatch complains that adt7410_update_device() can return error pointers.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
---

diff --git a/drivers/hwmon/adt7410.c b/drivers/hwmon/adt7410.c
index f8606dff3d1c..030c8d7c33a5 100644
--- a/drivers/hwmon/adt7410.c
+++ b/drivers/hwmon/adt7410.c
@@ -236,9 +236,14 @@ static ssize_t adt7410_show_t_hyst(struct device *dev,
 				   char *buf)
 {
 	struct sensor_device_attribute *attr = to_sensor_dev_attr(da);
-	struct adt7410_data *data = adt7410_update_device(dev);
+	struct adt7410_data *data;
 	int nr = attr->index;
-	int hyst = (data->hyst & ADT7410_T_HYST_MASK) * 1000;
+	int hyst;
+
+	data = adt7410_update_device(dev);
+	if (IS_ERR(data))
+		return PTR_ERR(data);
+	hyst = (data->hyst & ADT7410_T_HYST_MASK) * 1000;
 
 	/*
 	 * hysteresis is stored as a 4 bit offset in the device, convert it