diff options
| author | Alexey Khoroshilov <khoroshilov@ispras.ru> | 2013-10-07 18:06:04 -0300 |
|---|---|---|
| committer | Mauro Carvalho Chehab <m.chehab@samsung.com> | 2013-10-17 06:05:28 -0300 |
| commit | 256d013a9bcc9a39b2e4b34ab19219bd054cf270 (patch) | |
| tree | b16380050dbe612e9c3a60e9f5bd5d6b006faaaf /include/linux/reservation.h | |
| parent | 1f61d41585f8a7a52d67fd3d6aa4aa6db75f360c (diff) | |
[media] cx231xx: fix double free and leaks on failure path in cx231xx_usb_probe()
There are numerous issues in error handling code of cx231xx initialization.
Double free (when cx231xx_init_dev() calls kfree(dev) via cx231xx_release_resources()
and then cx231xx_usb_probe() does the same) and memory leaks
(e.g. usb_get_dev() before (ifnum != 1) check in cx231xx_usb_probe())
are just a few of them.
The patch fixes the issues in cx231xx_usb_probe() and cx231xx_init_dev()
by moving usb_get_dev(interface_to_usbdev(interface)) below in code and
implementing proper error handling.
Found by Linux Driver Verification project (linuxtesting.org).
Signed-off-by: Alexey Khoroshilov <khoroshilov@ispras.ru>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Diffstat (limited to 'include/linux/reservation.h')
0 files changed, 0 insertions, 0 deletions