]> Pileus Git - ~andy/linux/commitdiff
[media] marvell-ccic: drop free_irq for devm_request_irq allocated irq
authorWei Yongjun <yongjun_wei@trendmicro.com.cn>
Tue, 24 Sep 2013 02:35:50 +0000 (23:35 -0300)
committerMauro Carvalho Chehab <m.chehab@samsung.com>
Thu, 17 Oct 2013 14:28:09 +0000 (11:28 -0300)
irq allocated with devm_request_irq should not be freed using
free_irq, because doing so causes a dangling pointer, and a
subsequent double free.

Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Acked-by: Jonathan Corbet <corbet@lwn.net>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
drivers/media/platform/marvell-ccic/mmp-driver.c

index b5a19af5c587103c1c134b8b5fd0e19a13dc8853..3458fa0e2fd537916270fd9fbb0908d2b1610821 100644 (file)
@@ -481,7 +481,6 @@ static int mmpcam_remove(struct mmp_camera *cam)
        struct mmp_camera_platform_data *pdata;
 
        mmpcam_remove_device(cam);
-       free_irq(cam->irq, mcam);
        mccic_shutdown(mcam);
        mmpcam_power_down(mcam);
        pdata = cam->pdev->dev.platform_data;