From ff6ee89494af40172628acd4b659c2ab5a93626b 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 c38add466..bd3b8288e 100644 --- a/dix/dispatch.c +++ b/dix/dispatch.c @@ -2605,13 +2605,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; }