fdo-mirrors/xserver
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/xorg/xserver.git synced 2026-02-15 11:30:30 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

xace: typesafe hook function for XACE_SERVER_ACCESS

Browse source 
he generic XaceHook() call isn't typesafe (und unnecessarily slow).
Better add an explicit function, just like we already have for others.

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1556>
(cherry picked from commit 47d6c3ad75)
This commit is contained in:
Enrico Weigelt, metux IT consult 2024-05-17 15:17:27 +02:00 • committed by Alan Coopersmith
parent 65c3afbb61
commit dd0dcbe99d
7 changed files with 16 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
Xext/xace.c
View file

@ -109,13 +109,19 @@ int XaceHookExtAccess(ClientPtr client, ExtensionEntry *ext)
return rec.status;
}
int XaceHookServerAccess(ClientPtr client, Mask access_mode)
{
XaceServerAccessRec rec = { client, access_mode, Success };
CallCallbacks(&XaceHooks[XACE_SERVER_ACCESS], &rec);
return rec.status;
}
/* Entry point for hook functions. Called by Xserver.
*/
int
XaceHook(int hook, ...)
{
union {
XaceServerAccessRec server;
XaceScreenAccessRec screen;
XaceAuthAvailRec auth;
XaceKeyAvailRec key;
@ -134,13 +140,6 @@ XaceHook(int hook, ...)
* sets calldata directly to a single argument (with no return result)
*/
switch (hook) {
case XACE_SERVER_ACCESS:
u.server.client = va_arg(ap, ClientPtr);
u.server.access_mode = va_arg(ap, Mask);
u.server.status = Success; /* default allow */
prv = &u.server.status;
break;
case XACE_SCREEN_ACCESS:
case XACE_SCREENSAVER_ACCESS:
u.screen.client = va_arg(ap, ClientPtr);

1
Xext/xace.h
View file

@ -92,6 +92,7 @@ int XaceHookSendAccess(ClientPtr client, DeviceIntPtr dev, WindowPtr win,
int XaceHookReceiveAccess(ClientPtr client, WindowPtr win, xEventPtr ev, int count);
int XaceHookClientAccess(ClientPtr client, ClientPtr target, Mask access_mode);
int XaceHookExtAccess(ClientPtr client, ExtensionEntry *ext);
int XaceHookServerAccess(ClientPtr client, Mask access_mode);
/* Register a callback for a given hook.

2
dix/dispatch.c
View file

@ -3275,7 +3275,7 @@ ProcListHosts(ClientPtr client)
REQUEST_SIZE_MATCH(xListHostsReq);
/* untrusted clients can't list hosts */
result = XaceHook(XACE_SERVER_ACCESS, client, DixReadAccess);
result = XaceHookServerAccess(client, DixReadAccess);
if (result != Success)
return result;

8
dix/dixfonts.c
View file

@ -805,7 +805,7 @@ ListFonts(ClientPtr client, unsigned char *pattern, unsigned length,
if (length > XLFDMAXFONTNAMELEN)
return BadAlloc;
i = XaceHook(XACE_SERVER_ACCESS, client, DixGetAttrAccess);
i = XaceHookServerAccess(client, DixGetAttrAccess);
if (i != Success)
return i;
@ -1051,7 +1051,7 @@ StartListFontsWithInfo(ClientPtr client, int length, unsigned char *pattern,
if (length > XLFDMAXFONTNAMELEN)
return BadAlloc;
i = XaceHook(XACE_SERVER_ACCESS, client, DixGetAttrAccess);
i = XaceHookServerAccess(client, DixGetAttrAccess);
if (i != Success)
return i;
@ -1674,7 +1674,7 @@ SetFontPathElements(int npaths, unsigned char *paths, int *bad, Bool persist)
int
SetFontPath(ClientPtr client, int npaths, unsigned char *paths)
{
int err = XaceHook(XACE_SERVER_ACCESS, client, DixManageAccess);
int err = XaceHookServerAccess(client, DixManageAccess);
if (err != Success)
return err;
@ -1762,7 +1762,7 @@ GetFontPath(ClientPtr client, int *count, int *length, unsigned char **result)
int len;
FontPathElementPtr fpe;
i = XaceHook(XACE_SERVER_ACCESS, client, DixGetAttrAccess);
i = XaceHookServerAccess(client, DixGetAttrAccess);
if (i != Success)
return i;

2
os/access.c
View file

@ -1292,7 +1292,7 @@ AuthorizedClient(ClientPtr client)
return Success;
/* untrusted clients can't change host access */
rc = XaceHook(XACE_SERVER_ACCESS, client, DixManageAccess);
rc = XaceHookServerAccess(client, DixManageAccess);
if (rc != Success)
return rc;

2
os/connection.c
View file

@ -862,7 +862,7 @@ OnlyListenToOneClient(ClientPtr client)
{
int rc;
rc = XaceHook(XACE_SERVER_ACCESS, client, DixGrabAccess);
rc = XaceHookServerAccess(client, DixGrabAccess);
if (rc != Success)
return rc;

2
xkb/xkb.c
View file

@ -6956,7 +6956,7 @@ ProcXkbSetDebuggingFlags(ClientPtr client)
REQUEST(xkbSetDebuggingFlagsReq);
REQUEST_AT_LEAST_SIZE(xkbSetDebuggingFlagsReq);
rc = XaceHook(XACE_SERVER_ACCESS, client, DixDebugAccess);
rc = XaceHookServerAccess(client, DixDebugAccess);
if (rc != Success)
return rc;