diff options
| author | Jiri Slaby <jslaby@suse.cz> | 2013-04-04 16:32:09 -0300 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2013-04-08 06:54:01 -0300 |
| commit | a9bd87c232b87014b20fdf52416f80c5c1e869fc (patch) | |
| tree | 9c5dfe0a29a80007dde51c6a09acdac3fbb0ffbd /drivers/media/rc/ttusbir.c | |
| parent | fde04ab95d43e55959f12b92711b0ca4fed40637 (diff) | |
[media] MEDIA: ttusbir, fix double free
rc_unregister_device already calls rc_free_device to free the passed
device. But in one of ttusbir's probe fail paths, we call
rc_unregister_device _and_ rc_free_device. This is wrong and results
in a double free.
Instead, set rc to NULL resulting in rc_free_device being a noop.
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/rc/ttusbir.c')
| -rw-r--r-- | drivers/media/rc/ttusbir.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/media/rc/ttusbir.c b/drivers/media/rc/ttusbir.c index cf0d47f57fb..891762d167e 100644 --- a/drivers/media/rc/ttusbir.c +++ b/drivers/media/rc/ttusbir.c @@ -347,6 +347,7 @@ static int ttusbir_probe(struct usb_interface *intf, return 0; out3: rc_unregister_device(rc); + rc = NULL; out2: led_classdev_unregister(&tt->led); out: |