diff options
author | Dmitry Torokhov <dtor@insightbb.com> | 2006-08-01 22:45:28 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2006-09-27 14:58:52 -0400 |
commit | 17efe155253e3f81caa6cc57ec2af1c128618698 (patch) | |
tree | 4075a7dad14e51334db498665d4a1993fd1f6ba1 /drivers/usb/storage | |
parent | bd35078f47f795569fae29b2ff187e6f921460ad (diff) |
USB: onetouch - handle errors from input_register_device()
Onetouch: handle errors from input_register_device()
Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/usb/storage')
-rw-r--r-- | drivers/usb/storage/onetouch.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/drivers/usb/storage/onetouch.c b/drivers/usb/storage/onetouch.c index 313920d980c9..f843a0bcf107 100644 --- a/drivers/usb/storage/onetouch.c +++ b/drivers/usb/storage/onetouch.c | |||
@@ -135,6 +135,7 @@ int onetouch_connect_input(struct us_data *ss) | |||
135 | struct usb_onetouch *onetouch; | 135 | struct usb_onetouch *onetouch; |
136 | struct input_dev *input_dev; | 136 | struct input_dev *input_dev; |
137 | int pipe, maxp; | 137 | int pipe, maxp; |
138 | int error = -ENOMEM; | ||
138 | 139 | ||
139 | interface = ss->pusb_intf->cur_altsetting; | 140 | interface = ss->pusb_intf->cur_altsetting; |
140 | 141 | ||
@@ -211,15 +212,18 @@ int onetouch_connect_input(struct us_data *ss) | |||
211 | ss->suspend_resume_hook = usb_onetouch_pm_hook; | 212 | ss->suspend_resume_hook = usb_onetouch_pm_hook; |
212 | #endif | 213 | #endif |
213 | 214 | ||
214 | input_register_device(onetouch->dev); | 215 | error = input_register_device(onetouch->dev); |
216 | if (error) | ||
217 | goto fail3; | ||
215 | 218 | ||
216 | return 0; | 219 | return 0; |
217 | 220 | ||
221 | fail3: usb_free_urb(onetouch->irq); | ||
218 | fail2: usb_buffer_free(udev, ONETOUCH_PKT_LEN, | 222 | fail2: usb_buffer_free(udev, ONETOUCH_PKT_LEN, |
219 | onetouch->data, onetouch->data_dma); | 223 | onetouch->data, onetouch->data_dma); |
220 | fail1: kfree(onetouch); | 224 | fail1: kfree(onetouch); |
221 | input_free_device(input_dev); | 225 | input_free_device(input_dev); |
222 | return -ENOMEM; | 226 | return error; |
223 | } | 227 | } |
224 | 228 | ||
225 | void onetouch_release_input(void *onetouch_) | 229 | void onetouch_release_input(void *onetouch_) |