diff options
author | Milan Svoboda <msvoboda@ra.rockwell.com> | 2006-08-09 01:14:43 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2006-09-27 14:58:54 -0400 |
commit | a94da8971e836f32315f8832b0bf3e88bee9efae (patch) | |
tree | 6c9fe746a803f26b5c853c7cbb646a867b3f13a4 /drivers/usb/gadget | |
parent | e22fc27c87b41bda1b0daf8436224b0f79853482 (diff) |
USB gadget: gadgetfs dont try to lock before free
I spotted this during my tests with -rt on arm. The -rt patch contains
some better tools
to diagnose problems with locks and some other things...
Original code tries to take semaphore in BUG_ON and then free the memory
with this semaphore.
Signed-off-by: Milan Svoboda <msvoboda@ra.rockwell.com>
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/usb/gadget')
-rw-r--r-- | drivers/usb/gadget/inode.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/usb/gadget/inode.c b/drivers/usb/gadget/inode.c index ed9b404e5f5a..a38b6af2c358 100644 --- a/drivers/usb/gadget/inode.c +++ b/drivers/usb/gadget/inode.c | |||
@@ -223,7 +223,6 @@ static void put_ep (struct ep_data *data) | |||
223 | /* needs no more cleanup */ | 223 | /* needs no more cleanup */ |
224 | BUG_ON (!list_empty (&data->epfiles)); | 224 | BUG_ON (!list_empty (&data->epfiles)); |
225 | BUG_ON (waitqueue_active (&data->wait)); | 225 | BUG_ON (waitqueue_active (&data->wait)); |
226 | BUG_ON (down_trylock (&data->lock) != 0); | ||
227 | kfree (data); | 226 | kfree (data); |
228 | } | 227 | } |
229 | 228 | ||