Skip to content

Commit d50ef35

Browse files
jhovoldgregkh
authored andcommitted
spi: topcliff-pch: fix use-after-free on unbind
commit 9d72732 upstream. Give the driver a chance to flush its queue before releasing the DMA buffers on driver unbind Fixes: c37f3c2 ("spi/topcliff_pch: DMA support") Cc: stable@vger.kernel.org # 3.1 Cc: Tomoya MORINAGA <tomoya-linux@dsn.okisemi.com> Signed-off-by: Johan Hovold <johan@kernel.org> Link: https://patch.msgid.link/20260414134319.978196-9-johan@kernel.org Signed-off-by: Mark Brown <broonie@kernel.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
1 parent b3bdc7a commit d50ef35

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

drivers/spi/spi-topcliff-pch.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1410,9 +1410,6 @@ static void pch_spi_pd_remove(struct platform_device *plat_dev)
14101410

14111411
spi_unregister_controller(data->host);
14121412

1413-
if (use_dma)
1414-
pch_free_dma_buf(board_dat, data);
1415-
14161413
/* check for any pending messages; no action is taken if the queue
14171414
* is still full; but at least we tried. Unload anyway */
14181415
count = 500;
@@ -1436,6 +1433,9 @@ static void pch_spi_pd_remove(struct platform_device *plat_dev)
14361433
free_irq(board_dat->pdev->irq, data);
14371434
}
14381435

1436+
if (use_dma)
1437+
pch_free_dma_buf(board_dat, data);
1438+
14391439
pci_iounmap(board_dat->pdev, data->io_remap_addr);
14401440

14411441
spi_controller_put(data->host);

0 commit comments

Comments
 (0)