frederik
/
xserver
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

record: drop swapping request length fields 

The request struct's length fields aren't used anymore - we have the
client->req_len field instead, which also is bigreq-compatible.

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1639>
This commit is contained in:
Enrico Weigelt, metux IT consult 2024-08-06 14:38:46 +02:00 committed by Marge Bot
parent 9b88a64ba9
commit b4df6f1fe3
1 changed files with 0 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
record/record.c
View File

@ -2487,8 +2487,6 @@ static int _X_COLD
SProcRecordQueryVersion(ClientPtr client) SProcRecordQueryVersion(ClientPtr client)
{ {
REQUEST(xRecordQueryVersionReq); REQUEST(xRecordQueryVersionReq);
swaps(&stuff->length);
REQUEST_SIZE_MATCH(xRecordQueryVersionReq); REQUEST_SIZE_MATCH(xRecordQueryVersionReq);
swaps(&stuff->majorVersion); swaps(&stuff->majorVersion);
swaps(&stuff->minorVersion); swaps(&stuff->minorVersion);
@ -2525,7 +2523,6 @@ SProcRecordCreateContext(ClientPtr client)
REQUEST(xRecordCreateContextReq); REQUEST(xRecordCreateContextReq);
int status; int status;
swaps(&stuff->length);
REQUEST_AT_LEAST_SIZE(xRecordCreateContextReq); REQUEST_AT_LEAST_SIZE(xRecordCreateContextReq);
if ((status = SwapCreateRegister(client, (void *) stuff)) != Success) if ((status = SwapCreateRegister(client, (void *) stuff)) != Success)
return status; return status;
@ -2538,7 +2535,6 @@ SProcRecordRegisterClients(ClientPtr client)
REQUEST(xRecordRegisterClientsReq); REQUEST(xRecordRegisterClientsReq);
int status; int status;
swaps(&stuff->length);
REQUEST_AT_LEAST_SIZE(xRecordRegisterClientsReq); REQUEST_AT_LEAST_SIZE(xRecordRegisterClientsReq);
if ((status = SwapCreateRegister(client, (void *) stuff)) != Success) if ((status = SwapCreateRegister(client, (void *) stuff)) != Success)
return status; return status;
@ -2549,8 +2545,6 @@ static int _X_COLD
SProcRecordUnregisterClients(ClientPtr client) SProcRecordUnregisterClients(ClientPtr client)
{ {
REQUEST(xRecordUnregisterClientsReq); REQUEST(xRecordUnregisterClientsReq);
swaps(&stuff->length);
REQUEST_AT_LEAST_SIZE(xRecordUnregisterClientsReq); REQUEST_AT_LEAST_SIZE(xRecordUnregisterClientsReq);
swapl(&stuff->context); swapl(&stuff->context);
swapl(&stuff->nClients); swapl(&stuff->nClients);
@ -2562,8 +2556,6 @@ static int _X_COLD
SProcRecordGetContext(ClientPtr client) SProcRecordGetContext(ClientPtr client)
{ {
REQUEST(xRecordGetContextReq); REQUEST(xRecordGetContextReq);
swaps(&stuff->length);
REQUEST_SIZE_MATCH(xRecordGetContextReq); REQUEST_SIZE_MATCH(xRecordGetContextReq);
swapl(&stuff->context); swapl(&stuff->context);
return ProcRecordGetContext(client); return ProcRecordGetContext(client);
@ -2573,8 +2565,6 @@ static int _X_COLD
SProcRecordEnableContext(ClientPtr client) SProcRecordEnableContext(ClientPtr client)
{ {
REQUEST(xRecordEnableContextReq); REQUEST(xRecordEnableContextReq);
swaps(&stuff->length);
REQUEST_SIZE_MATCH(xRecordEnableContextReq); REQUEST_SIZE_MATCH(xRecordEnableContextReq);
swapl(&stuff->context); swapl(&stuff->context);
return ProcRecordEnableContext(client); return ProcRecordEnableContext(client);
@ -2584,8 +2574,6 @@ static int _X_COLD
SProcRecordDisableContext(ClientPtr client) SProcRecordDisableContext(ClientPtr client)
{ {
REQUEST(xRecordDisableContextReq); REQUEST(xRecordDisableContextReq);
swaps(&stuff->length);
REQUEST_SIZE_MATCH(xRecordDisableContextReq); REQUEST_SIZE_MATCH(xRecordDisableContextReq);
swapl(&stuff->context); swapl(&stuff->context);
return ProcRecordDisableContext(client); return ProcRecordDisableContext(client);
@ -2595,8 +2583,6 @@ static int _X_COLD
SProcRecordFreeContext(ClientPtr client) SProcRecordFreeContext(ClientPtr client)
{ {
REQUEST(xRecordFreeContextReq); REQUEST(xRecordFreeContextReq);
swaps(&stuff->length);
REQUEST_SIZE_MATCH(xRecordFreeContextReq); REQUEST_SIZE_MATCH(xRecordFreeContextReq);
swapl(&stuff->context); swapl(&stuff->context);
return ProcRecordFreeContext(client); return ProcRecordFreeContext(client);