From 27364a0cc5ba56abcf53a51d3bbed89e0775ec36 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 b6afe230e..fa1f9adbb 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;