FAILED: patch "[PATCH] iommufd: Fix return value of iommufd_fault_fops_write()" failed to apply to 6.12-stable tree
Subsystems (3)
[email protected]
maintainerThe patch below does not apply to the 6.12-stable tree.
If someone wants it applied there, or to any other stable or longterm tree, then please email the backport, including the original git commit id to <[email protected]>.
To reproduce the conflict and resubmit, you may use the following commands:
git fetch https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/ linux-6.12.y
git checkout FETCH_HEAD
git cherry-pick -x aaca2aa92785a6ab8e3183e7184bca447a99cd76
# <resolve conflicts, build, test, etc.>
git commit -s
git send-email --to '<[email protected]>' --in-reply-to '2026051240-paralyses-sprawl-a7f1@gregkh' --subject-prefix 'PATCH 6.12.y' HEAD^..
Possible dependencies:
thanks,
greg k-h
------------------ original commit in Linus's tree ------------------
From aaca2aa92785a6ab8e3183e7184bca447a99cd76 Mon Sep 17 00:00:00 2001
From: Zhenzhong Duan <[email protected]>
Date: Sun, 29 Mar 2026 23:07:55 -0400
Subject: [PATCH] iommufd: Fix return value of iommufd_fault_fops_write()
copy_from_user() may return number of bytes failed to copy, we should not pass over this number to user space to cheat that write() succeed. Instead, -EFAULT should be returned.
drivers/iommu/iommufd/eventq.c +3 -2
@@ -187,9 +187,10 @@ static ssize_t iommufd_fault_fops_write(struct file *filep, const char __user *bmutex_lock (&fault->mutex);while (count > done) { - rc =copy_from_user (&response, buf + done, response_size); -if (rc) +if (copy_from_user (&response, buf + done, response_size)) { + rc = -EFAULT ;break ; + }static_assert ((int )IOMMUFD_PAGE_RESP_SUCCESS == (int )IOMMU_PAGE_RESP_SUCCESS );
Sasha Levin
maintainerPATCH 1/1 iommufd: Fix return value of iommufd_fault_fops_write()
From: Zhenzhong Duan <[email protected]>
[ Upstream commit aaca2aa92785a6ab8e3183e7184bca447a99cd76 ]
copy_from_user() may return number of bytes failed to copy, we should not pass over this number to user space to cheat that write() succeed. Instead, -EFAULT should be returned.
--- drivers/iommu/iommufd/fault.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)
drivers/iommu/iommufd/fault.c +3 -2
@@ -317,9 +317,10 @@ static ssize_t iommufd_fault_fops_write(struct file *filep, const char __user *bmutex_lock (&fault->mutex);while (count > done) { - rc =copy_from_user (&response, buf + done, response_size); -if (rc) +if (copy_from_user (&response, buf + done, response_size)) { + rc = -EFAULT ;break ; + }static_assert ((int )IOMMUFD_PAGE_RESP_SUCCESS == (int )IOMMU_PAGE_RESP_SUCCESS );
Signature · 1 line
2.53.0