aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/hid
diff options
context:
space:
mode:
Diffstat (limited to 'drivers/hid')
-rw-r--r--drivers/hid/hidraw.c14
1 files changed, 10 insertions, 4 deletions
diff --git a/drivers/hid/hidraw.c b/drivers/hid/hidraw.c
index 20ebba1032b8..8503197a8131 100644
--- a/drivers/hid/hidraw.c
+++ b/drivers/hid/hidraw.c
@@ -282,7 +282,7 @@ EXPORT_SYMBOL_GPL(hidraw_report_event);
282 282
283int hidraw_connect(struct hid_device *hid) 283int hidraw_connect(struct hid_device *hid)
284{ 284{
285 int minor, result = -EINVAL; 285 int minor, result;
286 struct hidraw *dev; 286 struct hidraw *dev;
287 287
288 /* TODO currently we accept any HID device. This should later 288 /* TODO currently we accept any HID device. This should later
@@ -290,8 +290,11 @@ int hidraw_connect(struct hid_device *hid)
290 * non-input applications 290 * non-input applications
291 */ 291 */
292 292
293 if (!(dev = kzalloc(sizeof(struct hidraw), GFP_KERNEL))) 293 dev = kzalloc(sizeof(struct hidraw), GFP_KERNEL);
294 return -1; 294 if (!dev)
295 return -ENOMEM;
296
297 result = -EINVAL;
295 298
296 spin_lock(&minors_lock); 299 spin_lock(&minors_lock);
297 300
@@ -305,8 +308,10 @@ int hidraw_connect(struct hid_device *hid)
305 308
306 spin_unlock(&minors_lock); 309 spin_unlock(&minors_lock);
307 310
308 if (result) 311 if (result) {
312 kfree(dev);
309 goto out; 313 goto out;
314 }
310 315
311 dev->dev = device_create(hidraw_class, NULL, MKDEV(hidraw_major, minor), 316 dev->dev = device_create(hidraw_class, NULL, MKDEV(hidraw_major, minor),
312 "%s%d", "hidraw", minor); 317 "%s%d", "hidraw", minor);
@@ -316,6 +321,7 @@ int hidraw_connect(struct hid_device *hid)
316 hidraw_table[minor] = NULL; 321 hidraw_table[minor] = NULL;
317 spin_unlock(&minors_lock); 322 spin_unlock(&minors_lock);
318 result = PTR_ERR(dev->dev); 323 result = PTR_ERR(dev->dev);
324 kfree(dev);
319 goto out; 325 goto out;
320 } 326 }
321 327