summaryrefslogtreecommitdiff
path: root/drivers/media/rc/ttusbir.c
diff options
context:
space:
mode:
authorJiri Slaby <jslaby@suse.cz>2013-04-04 23:32:09 +0400
committerMauro Carvalho Chehab <mchehab@redhat.com>2013-04-08 13:54:01 +0400
commita9bd87c232b87014b20fdf52416f80c5c1e869fc (patch)
tree9c5dfe0a29a80007dde51c6a09acdac3fbb0ffbd /drivers/media/rc/ttusbir.c
parentfde04ab95d43e55959f12b92711b0ca4fed40637 (diff)
downloadlinux-a9bd87c232b87014b20fdf52416f80c5c1e869fc.tar.xz
[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.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/media/rc/ttusbir.c b/drivers/media/rc/ttusbir.c
index cf0d47f57fb2..891762d167ed 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: