diff options
author | Andrew Morton <akpm@linux-foundation.org> | 2007-12-13 19:15:34 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2008-02-01 17:34:57 -0500 |
commit | 26e109b0a90b5ef7b025e44a969836ab9a598ffc (patch) | |
tree | df2117de26c53ab4137663107c699963968c20b9 /drivers/usb | |
parent | 1cfab028afe5827433195c57b6f36d61a79beb50 (diff) |
USB: testing driver: don't free a locked mutex
Dopey thing to do and lockdep will (or should) warn.
Spotted by Daniel Walker.
Cc: Matthias Kaehlcke <matthias.kaehlcke@gmail.com>
Cc: Daniel Walker <dwalker@mvista.com>
Cc: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/usb')
-rw-r--r-- | drivers/usb/misc/usbtest.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/usb/misc/usbtest.c b/drivers/usb/misc/usbtest.c index e3851359d653..f1b886c204c0 100644 --- a/drivers/usb/misc/usbtest.c +++ b/drivers/usb/misc/usbtest.c | |||
@@ -1992,8 +1992,6 @@ static void usbtest_disconnect (struct usb_interface *intf) | |||
1992 | { | 1992 | { |
1993 | struct usbtest_dev *dev = usb_get_intfdata (intf); | 1993 | struct usbtest_dev *dev = usb_get_intfdata (intf); |
1994 | 1994 | ||
1995 | mutex_lock(&dev->lock); | ||
1996 | |||
1997 | usb_set_intfdata (intf, NULL); | 1995 | usb_set_intfdata (intf, NULL); |
1998 | dev_dbg (&intf->dev, "disconnect\n"); | 1996 | dev_dbg (&intf->dev, "disconnect\n"); |
1999 | kfree (dev); | 1997 | kfree (dev); |