spi: bcm2835: Add spi_master_get() call to prevent use after free
authorAxel Lin <axel.lin@ingics.com>
Thu, 22 Aug 2013 15:19:07 +0000 (23:19 +0800)
committerMark Brown <broonie@linaro.org>
Thu, 22 Aug 2013 18:07:51 +0000 (19:07 +0100)
The call to spi_unregister_master results in device memory being freed, it must
no longer be accessed afterwards. Thus call spi_master_get() to get an extra
reference to the device and call spi_master_put() only after the last access to
device data.

Note, current code has an extra spi_master_put() call in bcm2835_spi_remove().
Thus this patch just adds an spi_master_get() to balance the reference count.

Signed-off-by: Axel Lin <axel.lin@ingics.com>
Signed-off-by: Mark Brown <broonie@linaro.org>
drivers/spi/spi-bcm2835.c

index 7de617aba6955a122341966b8128ecb31f8df68a..52c81481c5c77408e90e22bbf2580e8041cf4a2c 100644 (file)
@@ -377,7 +377,7 @@ out_master_put:
 
 static int bcm2835_spi_remove(struct platform_device *pdev)
 {
-       struct spi_master *master = platform_get_drvdata(pdev);
+       struct spi_master *master = spi_master_get(platform_get_drvdata(pdev));
        struct bcm2835_spi *bs = spi_master_get_devdata(master);
 
        free_irq(bs->irq, master);