[xen staging-4.17] gnttab: move "only local domain may copy by frame" check

[patchbot@xxxxxxx]

commit 2445b30a442ff3554ec477d5ec6445f92634394a
Author:     Jan Beulich <jbeulich@xxxxxxxx>
AuthorDate: Tue Nov 3 12:07:06 2026 +0000
Commit:     Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
CommitDate: Tue Nov 3 12:08:17 2026 +0000

    gnttab: move "only local domain may copy by frame" check
    
    gnttab_copy_one() calls gnttab_copy_lock_domain{s,}() only conditionally.
    The non-GREF check towards the requestor being DOMID_SELF, however, needs
    doing on every batch element. Move the check into gnttab_copy_claim_buf()
    to achieve that.
    
    This is CVE-2026-102704, part of XSA-523.
    
    Fixes: d28f42f2703e ("grant-table: defer releasing pages acquired in a 
grant copy")
    Assisted-by: Claude
    Reported-by: Stephan Lipp <Stephan@xxxxxxxxxxxxx>
    Reported-by: CodeAnt AI Security Research <securityresearch@xxxxxxxxxx>
    Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
    Reviewed-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
    (cherry picked from commit cd4d68f86cb111e2ea0a17cfb4fb8cd9c6c69e83)
---
 xen/common/grant_table.c | 16 ++++++----------
 1 file changed, 6 insertions(+), 10 deletions(-)

diff --git a/xen/common/grant_table.c b/xen/common/grant_table.c
index 682fa9af74..c190fb7f4c 100644
--- a/xen/common/grant_table.c
+++ b/xen/common/grant_table.c
@@ -2887,13 +2887,8 @@ struct gnttab_copy_buf {
     bool_t have_type;
 };
 
-static int gnttab_copy_lock_domain(domid_t domid, bool is_gref,
-                                   struct gnttab_copy_buf *buf)
+static int gnttab_copy_lock_domain(domid_t domid, struct gnttab_copy_buf *buf)
 {
-    /* Only DOMID_SELF may reference via frame. */
-    if ( domid != DOMID_SELF && !is_gref )
-        return GNTST_permission_denied;
-
     buf->domain = rcu_lock_domain_by_any_id(domid);
 
     if ( !buf->domain )
@@ -2925,12 +2920,10 @@ static int gnttab_copy_lock_domains(const struct 
gnttab_copy *op,
 {
     int rc;
 
-    rc = gnttab_copy_lock_domain(op->source.domid,
-                                 op->flags & GNTCOPY_source_gref, src);
+    rc = gnttab_copy_lock_domain(op->source.domid, src);
     if ( rc < 0 )
         goto error;
-    rc = gnttab_copy_lock_domain(op->dest.domid,
-                                 op->flags & GNTCOPY_dest_gref, dest);
+    rc = gnttab_copy_lock_domain(op->dest.domid, dest);
     if ( rc < 0 )
         goto error;
 
@@ -2993,6 +2986,9 @@ static int gnttab_copy_claim_buf(const struct gnttab_copy 
*op,
         buf->ptr.u.ref = ptr->u.ref;
         buf->have_grant = 1;
     }
+    /* Only DOMID_SELF may reference via frame. */
+    else if ( ptr->domid != DOMID_SELF )
+        return GNTST_permission_denied;
     else
     {
         rc = get_paged_frame(ptr->u.gmfn, &buf->mfn, &buf->page,
--
generated by git-patchbot for /home/xen/git/xen.git#staging-4.17