spi: rspi: Add spi_master_get() call to prevent use after free
authorAxel Lin <axel.lin@ingics.com>
Sat, 31 Aug 2013 11:42:56 +0000 (19:42 +0800)
committerMark Brown <broonie@linaro.org>
Sat, 31 Aug 2013 11:58:53 +0000 (12:58 +0100)
In rspi_remove(), current code dereferences rspi after spi_unregister_master(),
thus add an extra spi_master_get() call is necessary to prevent use after free.

Current code already has an extra spi_master_put() call in rspi_remove(), so
this patch just adds a spi_master_get() call rather than a spi_master_get() with
spi_master_put() calls.

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

index 00c32320dce87c0adb66eca0eef65e9d2c417e2c..49ae72a93087a15ab9c4b08e0a17f51d8d9360dc 100644 (file)
@@ -726,7 +726,7 @@ static void rspi_release_dma(struct rspi_data *rspi)
 
 static int rspi_remove(struct platform_device *pdev)
 {
-       struct rspi_data *rspi = platform_get_drvdata(pdev);
+       struct rspi_data *rspi = spi_master_get(platform_get_drvdata(pdev));
 
        spi_unregister_master(rspi->master);
        rspi_release_dma(rspi);