diff options
author | Jesper Juhl <jj@chaosbits.net> | 2008-11-25 08:57:54 -0500 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2012-12-21 13:28:58 -0500 |
commit | bbe2a1d32f40c01ca1a7e7795e20ca06f87ffc9b (patch) | |
tree | 1f448364296d0be3d7ddb15f1fc31e1c8abef708 | |
parent | e5d85b9ac3133f67460ea5b2d4e33e0473d6eb4b (diff) |
[media] rc: Fix double free in gpio_ir_recv_remove()
Since rc_unregister_device() frees its argument there's no need to
subsequently call rc_free_device() on the same variable - in fact it's
a double free bug.
Easily fixed by just removing the rc_free_device() call.
Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
-rw-r--r-- | drivers/media/rc/gpio-ir-recv.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/media/rc/gpio-ir-recv.c b/drivers/media/rc/gpio-ir-recv.c index 32db5f59fbc3..03e3cf6eb68f 100644 --- a/drivers/media/rc/gpio-ir-recv.c +++ b/drivers/media/rc/gpio-ir-recv.c | |||
@@ -148,7 +148,6 @@ static int __devexit gpio_ir_recv_remove(struct platform_device *pdev) | |||
148 | platform_set_drvdata(pdev, NULL); | 148 | platform_set_drvdata(pdev, NULL); |
149 | rc_unregister_device(gpio_dev->rcdev); | 149 | rc_unregister_device(gpio_dev->rcdev); |
150 | gpio_free(gpio_dev->gpio_nr); | 150 | gpio_free(gpio_dev->gpio_nr); |
151 | rc_free_device(gpio_dev->rcdev); | ||
152 | kfree(gpio_dev); | 151 | kfree(gpio_dev); |
153 | return 0; | 152 | return 0; |
154 | } | 153 | } |