From 2f89d4fed63a6ea065100e3ed6f9e13bad4113bb 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 22c6c18387..425d855345 100644 --- a/dix/colormap.c +++ b/dix/colormap.c @@ -1025,7 +1025,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, @@ -1043,7 +1043,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;