drivers/staging/iio: call mutex_unlock in error handling code
authorJulia Lawall <julia@diku.dk>
Wed, 4 Aug 2010 19:54:26 +0000 (21:54 +0200)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 31 Aug 2010 18:40:35 +0000 (11:40 -0700)
Adjust the error handling code so that it benefits from the call to
mutex_unlock at the end of the function.

The semantic match that finds this problem is as follows:
(http://coccinelle.lip6.fr/)

// <smpl>
@rcu exists@
position p1;
expression E;
@@

mutex_lock@p1(E);
...
mutex_unlock(E);

@exists@
position rcu.p1;
expression E;
@@

*mutex_lock@p1(E);
... when != mutex_unlock(E);
?*return ...;
// </smpl>

Signed-off-by: Julia Lawall <julia@diku.dk>
Acked-by: Jonathan Cameron <jic23@cam.ac.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/iio/magnetometer/hmc5843.c

index 92f6c6fb90fee13e89a9d23bd109e0e1e152094c..ea5f749c5fe771524eb4ebc857f97630e1e70226 100644 (file)
@@ -220,11 +220,15 @@ static ssize_t hmc5843_set_operating_mode(struct device *dev,
        int error;
        mutex_lock(&data->lock);
        error = strict_strtoul(buf, 10, &operating_mode);
-       if (error)
-               return error;
+       if (error) {
+               count = error;
+               goto exit;
+       }
        dev_dbg(dev, "set Conversion mode to %lu\n", operating_mode);
-       if (operating_mode > MODE_SLEEP)
-                       return -EINVAL;
+       if (operating_mode > MODE_SLEEP) {
+               count = -EINVAL;
+               goto exit;
+       }
 
        status = i2c_smbus_write_byte_data(client, this_attr->address,
                                        operating_mode);
@@ -437,18 +441,23 @@ static ssize_t set_range(struct device *dev,
        int error;
        mutex_lock(&data->lock);
        error = strict_strtoul(buf, 10, &range);
-       if (error)
-               return error;
+       if (error) {
+               count = error;
+               goto exit;
+       }
        dev_dbg(dev, "set range to %lu\n", range);
 
-       if (range > RANGE_6_5)
-               return -EINVAL;
+       if (range > RANGE_6_5) {
+               count = -EINVAL;
+               goto exit;
+       }
 
        data->range = range;
        range = range << RANGE_GAIN_OFFSET;
        if (i2c_smbus_write_byte_data(client, this_attr->address, range))
                count = -EINVAL;
 
+exit:
        mutex_unlock(&data->lock);
        return count;