Skip to content

Commit ee52da0

Browse files
jhovoldgregkh
authored andcommitted
spi: mpc52xx: fix use-after-free on unbind
commit 706b3dc upstream. The state machine work is scheduled by the interrupt handler and therefore needs to be cancelled after disabling interrupts to avoid a potential use-after-free. Fixes: 9848366 ("spi: mpc52xx: Add cancel_work_sync before module remove") Cc: stable@vger.kernel.org Cc: Pei Xiao <xiaopei01@kylinos.cn> Signed-off-by: Johan Hovold <johan@kernel.org> Link: https://patch.msgid.link/20260414134319.978196-5-johan@kernel.org Signed-off-by: Mark Brown <broonie@kernel.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
1 parent a3669f6 commit ee52da0

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

drivers/spi/spi-mpc52xx.c

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -525,10 +525,11 @@ static void mpc52xx_spi_remove(struct platform_device *op)
525525

526526
spi_unregister_controller(host);
527527

528-
cancel_work_sync(&ms->work);
529528
free_irq(ms->irq0, ms);
530529
free_irq(ms->irq1, ms);
531530

531+
cancel_work_sync(&ms->work);
532+
532533
for (i = 0; i < ms->gpio_cs_count; i++)
533534
gpiod_put(ms->gpio_cs[i]);
534535

0 commit comments

Comments
 (0)