diff options
author | Jiri Slaby <jirislaby@gmail.com> | 2009-03-11 21:47:38 +0100 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2009-03-17 14:01:29 -0700 |
commit | 46c9844c4014be53c67622dcd3ba4302f36e9cac (patch) | |
tree | eb02befaf420ec7e4da6848cd6d48321f759d3ad /drivers/usb/misc/adutux.c | |
parent | 909b6c3fc20ea772dc63a03986d74148fcbb1a1d (diff) | |
download | op-kernel-dev-46c9844c4014be53c67622dcd3ba4302f36e9cac.zip op-kernel-dev-46c9844c4014be53c67622dcd3ba4302f36e9cac.tar.gz |
USB: misc/adutux, fix lock imbalance
Don't unlock adutux_mutex when not held.
Signed-off-by: Jiri Slaby <jirislaby@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/usb/misc/adutux.c')
-rw-r--r-- | drivers/usb/misc/adutux.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/usb/misc/adutux.c b/drivers/usb/misc/adutux.c index 7b6922e..2035265 100644 --- a/drivers/usb/misc/adutux.c +++ b/drivers/usb/misc/adutux.c @@ -376,7 +376,7 @@ static int adu_release(struct inode *inode, struct file *file) if (dev->open_count <= 0) { dbg(1," %s : device not opened", __func__); retval = -ENODEV; - goto exit; + goto unlock; } adu_release_internal(dev); @@ -385,9 +385,9 @@ static int adu_release(struct inode *inode, struct file *file) if (!dev->open_count) /* ... and we're the last user */ adu_delete(dev); } - -exit: +unlock: mutex_unlock(&adutux_mutex); +exit: dbg(2," %s : leave, return value %d", __func__, retval); return retval; } |