From dc0630df2f55f3193095a5ac0cecfb6bc4bcca2e Mon Sep 17 00:00:00 2001 From: "Enrico Weigelt, metux IT consult" Date: Mon, 10 Mar 2025 20:00:00 +0100 Subject: [PATCH] dix: AllocColor() dont fire Xace hook when looking up color at root window This is creating false alarms that are complicated to catch, thus passing NULL client so the Xace hook in dixLookupResourceByType() doesn't fire here. Signed-off-by: Enrico Weigelt, metux IT consult --- dix/colormap.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dix/colormap.c b/dix/colormap.c index 9a8be28e2..bc7a5938f 100644 --- a/dix/colormap.c +++ b/dix/colormap.c @@ -1026,7 +1026,7 @@ AllocColor(ColormapPtr pmap, dixLookupResourceByType((void **) &prootmap, pmap->pScreen->defColormap, X11_RESTYPE_COLORMAP, - clients[client], DixReadAccess); + NULL, DixReadAccess); if (pmap->class == prootmap->class) FindColorInRootCmap(prootmap, prootmap->red, entries, &rgb, @@ -1044,7 +1044,7 @@ AllocColor(ColormapPtr pmap, dixLookupResourceByType((void **) &prootmap, pmap->pScreen->defColormap, X11_RESTYPE_COLORMAP, - clients[client], DixReadAccess); + NULL, DixReadAccess); if (pmap->class == prootmap->class) { pixR = (*pPix & pVisual->redMask) >> pVisual->offsetRed;