diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-07-11 02:34:00 -0400 |
---|---|---|
committer | Jonathan Cameron <jic23@kernel.org> | 2012-07-12 15:00:20 -0400 |
commit | e59b9afecf7ddf52d31d24e800b473f19c23cfe0 (patch) | |
tree | 812dfea07e848bc7b308cdbd50a396689a86705b /drivers/iio | |
parent | 21fa54e4017fb0ed2107282e6057528ca59b8942 (diff) |
iio: double unlock on error path
We should be holding the mutex when we goto error_free_chans.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Diffstat (limited to 'drivers/iio')
-rw-r--r-- | drivers/iio/inkern.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/iio/inkern.c b/drivers/iio/inkern.c index e2aded04996b..b5afc2ff34fd 100644 --- a/drivers/iio/inkern.c +++ b/drivers/iio/inkern.c | |||
@@ -198,11 +198,12 @@ struct iio_channel *iio_channel_get_all(const char *name) | |||
198 | iio_device_get(chans[mapind].indio_dev); | 198 | iio_device_get(chans[mapind].indio_dev); |
199 | mapind++; | 199 | mapind++; |
200 | } | 200 | } |
201 | mutex_unlock(&iio_map_list_lock); | ||
202 | if (mapind == 0) { | 201 | if (mapind == 0) { |
203 | ret = -ENODEV; | 202 | ret = -ENODEV; |
204 | goto error_free_chans; | 203 | goto error_free_chans; |
205 | } | 204 | } |
205 | mutex_unlock(&iio_map_list_lock); | ||
206 | |||
206 | return chans; | 207 | return chans; |
207 | 208 | ||
208 | error_free_chans: | 209 | error_free_chans: |