diff options
author | Benjamin Tissoires <benjamin.tissoires@enac.fr> | 2011-02-15 15:41:10 +0100 |
---|---|---|
committer | Jiri Kosina <jkosina@suse.cz> | 2011-02-15 23:47:05 +0100 |
commit | d9236303d0b7ba8bbaeb6adbbf088c3fe2a9ab9e (patch) | |
tree | 24e2ca2a398484c986ab3b991a3487ac4c09f518 /drivers/hid/hid-input.c | |
parent | HID: Add teletext/color keys - gyration remote - EU version (GYAR3101CKDE) (diff) | |
download | linux-d9236303d0b7ba8bbaeb6adbbf088c3fe2a9ab9e.tar.xz linux-d9236303d0b7ba8bbaeb6adbbf088c3fe2a9ab9e.zip |
HID: kernel oops in out_cleanup in function hidinput_connect
Goto out_cleanup infers a kernel oops: hidinput_disconnect calls
input_unregister_driver to all members of hid->inputs.
However, hidinput already has been added to hid->inputs even
though input_register_device was not called.
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@enac.fr>
Reviewed-by: Dmitry Torokhov <dtor@mail.ru>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Diffstat (limited to 'drivers/hid/hid-input.c')
-rw-r--r-- | drivers/hid/hid-input.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/hid/hid-input.c b/drivers/hid/hid-input.c index 7f552bfad32c..f53911dcdc2d 100644 --- a/drivers/hid/hid-input.c +++ b/drivers/hid/hid-input.c @@ -928,6 +928,7 @@ int hidinput_connect(struct hid_device *hid, unsigned int force) return 0; out_cleanup: + list_del(&hidinput->list); input_free_device(hidinput->input); kfree(hidinput); out_unwind: |