From a5074bb749384e2f6365dfaba2ad44919e70b317 Mon Sep 17 00:00:00 2001 From: "Enrico Weigelt, metux IT consult" Date: Fri, 4 Apr 2025 19:33:30 +0200 Subject: [PATCH] dix: write out X_ListInstalledColormaps reply directly No need for using a complex callback machinery, if we just move the little pieces of byte-swapping directly into the request handler. Signed-off-by: Enrico Weigelt, metux IT consult --- dix/dispatch.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/dix/dispatch.c b/dix/dispatch.c index 27af7a120..e5311b3fd 100644 --- a/dix/dispatch.c +++ b/dix/dispatch.c @@ -2619,13 +2619,11 @@ ProcListInstalledColormaps(ClientPtr client) swaps(&rep.sequenceNumber); swapl(&rep.length); swaps(&rep.nColormaps); + SwapLongs(cm, nummaps * sizeof(Colormap) / 4); } WriteToClient(client, sizeof(rep), &rep); - if (client->swapped) - Swap32Write(client, nummaps * sizeof(Colormap), cm); - else - WriteToClient(client, nummaps * sizeof(Colormap), cm); + WriteToClient(client, nummaps * sizeof(Colormap), cm); free(cm); return Success; }