xserver: Branch 'master'

[GitLab Mirror]

 Xext/sync.c |    1 +
 1 file changed, 1 insertion(+)

New commits:
commit e6573baa7d99a77f44229b9a96a41bbda57e2387
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Thu Mar 21 18:08:35 2024 -0700

    Xext: SProcSyncCreateFence needs to swap drawable id too
    
    Otherwise it causes the server to return BadDrawable giving a
    byte-swapped resource id instead of the real id the client sent.
    
    Reported-by: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=69762
    Fixes: 397dfd9f8 ("Create/Destroy/Trigger/Reset/Query Fence Sync objs")
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
    Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1437>

diff --git a/Xext/sync.c b/Xext/sync.c
index 262f3f3c0..8da5bf616 100644
--- a/Xext/sync.c
+++ b/Xext/sync.c
@@ -2319,6 +2319,7 @@ SProcSyncCreateFence(ClientPtr client)
     REQUEST(xSyncCreateFenceReq);
     swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xSyncCreateFenceReq);
+    swapl(&stuff->d);
     swapl(&stuff->fid);
 
     return ProcSyncCreateFence(client);