hashirama/xserver
Watch
0
0
Fork
You've already forked xserver
 0
mirror of https://github.com/X11Libre/xserver.git synced 2026-11-15 05:23:40 +00:00
Code Issues Projects Releases Wiki Activity

Xext: security: use new X_REPLY_FIELD_* macros
Some checks are pending
Build X servers / xserver-build-ubuntu (push) Waiting to run
Details
Build X servers / drivers-build-ubuntu (push) Waiting to run
Details
Build X servers / xserver-build-mingw32-ubuntu (push) Waiting to run
Details
Build X servers / xserver-build-macos (push) Waiting to run
Details
Build X servers / xserver-build-freebsd (push) Waiting to run
Details
Build X servers / xserver-build-dragonflybsd (push) Waiting to run
Details
Build X servers / xserver-build-netbsd (push) Waiting to run
Details
Build X servers / xserver-build-cygwin (push) Waiting to run
Details
Build X servers / Check Signed-Off-By (push) Waiting to run
Details
Build X servers / Release pushed tag (push) Blocked by required conditions
Details
check PRs for merge conflicts / notify (push) Waiting to run
Details

Browse source 
Use the new X_REPLY_FIELD_* macros for reply byte-swapping.

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
This commit is contained in:
Enrico Weigelt, metux IT consult 2024-07-04 01:51:31 +02:00 • committed by Enrico Weigelt
commit 6445d434ed
1 changed files with 5 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

14
Xext/security.c
View file

@ -361,10 +361,8 @@ ProcSecurityQueryVersion(ClientPtr client)
.minorVersion = SERVER_SECURITY_MINOR_VERSION
};
if (client->swapped) {
swaps(&reply.majorVersion);
swaps(&reply.minorVersion);
}
X_REPLY_FIELD_CARD16(majorVersion);
X_REPLY_FIELD_CARD16(minorVersion);
return X_SEND_REPLY_SIMPLE(client, reply);
} /* ProcSecurityQueryVersion */
@ -554,16 +552,14 @@ ProcSecurityGenerateAuthorization(ClientPtr client)
.dataLength = authdata_len
};
if (client->swapped) {
swapl(&reply.authId);
swaps(&reply.dataLength);
}
SecurityAudit
("client %d generated authorization %lu trust %d timeout %lu group %lu events %lu\n",
client->index, (unsigned long)pAuth->id, pAuth->trustLevel, (unsigned long)pAuth->timeout,
(unsigned long)pAuth->group, (unsigned long)eventMask);
X_REPLY_FIELD_CARD32(authId);
X_REPLY_FIELD_CARD16(dataLength);
/* the request succeeded; don't call RemoveAuthorization or free pAuth */
return X_SEND_REPLY_WITH_RPCBUF(client, reply, rpcbuf);