diff options
| author | Greg Kroah-Hartman <[email protected]> | 2022-01-27 08:16:38 +0100 |
|---|---|---|
| committer | Ulf Hansson <[email protected]> | 2022-01-31 15:36:34 +0100 |
| commit | bd2db32e7c3e35bd4d9b8bbff689434a50893546 (patch) | |
| tree | 46c9118321490fa8ce22831c7f8a1689f3277a46 | |
| parent | 379f56c24e698f14242f532b1d0a0f1747725e08 (diff) | |
moxart: fix potential use-after-free on remove path
It was reported that the mmc host structure could be accessed after it
was freed in moxart_remove(), so fix this by saving the base register of
the device and using it instead of the pointer dereference.
Cc: Ulf Hansson <[email protected]>
Cc: Xiyu Yang <[email protected]>
Cc: Xin Xiong <[email protected]>
Cc: Xin Tan <[email protected]>
Cc: Tony Lindgren <[email protected]>
Cc: Yang Li <[email protected]>
Cc: [email protected]
Cc: stable <[email protected]>
Reported-by: whitehat002 <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Ulf Hansson <[email protected]>
| -rw-r--r-- | drivers/mmc/host/moxart-mmc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/mmc/host/moxart-mmc.c b/drivers/mmc/host/moxart-mmc.c index 16d1c7a43d33..b6eb75f4bbfc 100644 --- a/drivers/mmc/host/moxart-mmc.c +++ b/drivers/mmc/host/moxart-mmc.c @@ -705,12 +705,12 @@ static int moxart_remove(struct platform_device *pdev) if (!IS_ERR_OR_NULL(host->dma_chan_rx)) dma_release_channel(host->dma_chan_rx); mmc_remove_host(mmc); - mmc_free_host(mmc); writel(0, host->base + REG_INTERRUPT_MASK); writel(0, host->base + REG_POWER_CONTROL); writel(readl(host->base + REG_CLOCK_CONTROL) | CLK_OFF, host->base + REG_CLOCK_CONTROL); + mmc_free_host(mmc); return 0; } |