From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-alma10-1.taild15c8.ts.net [100.103.45.18]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 59EB945C70B; Tue, 25 Aug 2026 13:52:18 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=100.103.45.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787665939; cv=none; b=UOq7dt7WEpr+QDNjkZWnVmJ8D5RZG4mP0VMNacJLQd47hec0YzJrCTYSXFB22HfasFt/1Sxaa5oetCm6BvPpUzeXoSIUclU0xGDklM+lyyorraGI6UUa5+tj9iWqRiEcU+2+K4BymvlJxV1lOVls+iOmhDp5DZMmJTdw5xED7FM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787665939; c=relaxed/simple; bh=CLVx1GdRTtsKyekMD+CBptOo0Q9qwhH0VldrXfmmRdI=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=iaLlu3r4NfcGj5S3IEWy32RbNkXKJpb+tg/bf2O74Jtly3AYpoe2YGsBgS0smuhhzaEU3jv6YqVZOvZXIUJ6SNqNfiPHGkDyF4LiZbESQ7assY3fgQ7ToBL7+yWgsDN3BQHrbqXDvSOr6KeJ0mielrtqhTyt8kpj3/Vgnz78wzM= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b=QdkNz9ku; arc=none smtp.client-ip=100.103.45.18 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b="QdkNz9ku" Received: by smtp.kernel.org (Postfix) with ESMTPSA id B57E31F000E9; Tue, 25 Aug 2026 13:52:17 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linuxfoundation.org; s=korg; t=1787665938; bh=PhH5D9+l8GXb+c3HiEnbiB9nPVyVv7e4s4jIo8l1m+E=; h=From:To:Cc:Subject:Date:In-Reply-To:References; b=QdkNz9kubMpJ67utjm354zEMn1SJbAQN3UJprsgd85k9s7mdWOcV0SFDjrRxHgegz DCxw3vb3f98C7tkJgMQj4LADJA44UZ8L1JN2qoY9sr8bIW9e89d6B9qOT7eIa4F3cS J+akOPmBOigu5iYEvKDTgfiJfRR34PnX4vUQeECw= From: Greg Kroah-Hartman To: stable@vger.kernel.org Cc: Greg Kroah-Hartman , patches@lists.linux.dev, Pei Xiao , Ulf Hansson , Sasha Levin Subject: [PATCH 6.1 29/79] mmc: atmel-mci: Fix use-after-free in atmci_remove due to race condition Date: Tue, 25 Aug 2026 15:26:09 +0200 Message-ID: <20260825132542.841248617@linuxfoundation.org> X-Mailer: git-send-email 2.55.0 In-Reply-To: <20260825132541.677185791@linuxfoundation.org> References: <20260825132541.677185791@linuxfoundation.org> User-Agent: quilt/0.69 X-stable: review X-Patchwork-Hint: ignore Precedence: bulk X-Mailing-List: patches@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit 6.1-stable review patch. If anyone has any objections, please let me know. ------------------ From: Pei Xiao [ Upstream commit c125ee35a49a0518521b52b27631eef061b8719a ] In atmci_probe, &host->bh_work is bound with atmci_work_func, and atmci_interrupt, atmci_timeout_timer and atmci_dma_complete can all queue this work on system_bh_wq. If we remove the module, atmci_remove makes cleanup and the memory allocated for host with devm_kzalloc() is released after the remove callback returns, while the work mentioned above may still be pending or running. The sequence of operations that may lead to a UAF bug is as follows: CPU0 CPU1 | atmci_interrupt | queue_work(system_bh_wq, | &host->bh_work) atmci_remove | atmci_cleanup_slot(...) | atmci_writel(host, ATMCI_IDR, ~0UL) | timer_delete_sync(&host->timer) | dma_release_channel(host->dma.chan) | free_irq(platform_get_irq(pdev, 0), host) | | atmci_work_func | // use host // devm resources released after | // remove returns, host is freed | | // use host (use-after-free) Fix it by canceling the work after all the sources that can schedule it (IRQ handler, timeout timer and DMA completion callback) have been stopped, and before proceeding with the remaining cleanup in atmci_remove. Fixes: 7d2be0749a59 ("atmel-mci: Driver for Atmel on-chip MMC controllers") Assisted-by: Codex:deepseek-v4-flash Signed-off-by: Pei Xiao Cc: stable@vger.kernel.org Signed-off-by: Ulf Hansson Signed-off-by: Sasha Levin Signed-off-by: Greg Kroah-Hartman --- drivers/mmc/host/atmel-mci.c | 2 ++ 1 file changed, 2 insertions(+) --- a/drivers/mmc/host/atmel-mci.c +++ b/drivers/mmc/host/atmel-mci.c @@ -2628,6 +2628,8 @@ static int atmci_remove(struct platform_ free_irq(platform_get_irq(pdev, 0), host); + cancel_work_sync(&host->bh_work); + clk_disable_unprepare(host->mck); pm_runtime_disable(&pdev->dev);