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

(submit/fixup-req-len) 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>
This commit is contained in:
Enrico Weigelt, metux IT consult 2024-08-06 14:38:46 +02:00
parent eb6d712e31
commit 9a2cc5b639
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

@ -2488,8 +2488,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);
@ -2526,7 +2524,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;
@ -2539,7 +2536,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;
@ -2550,8 +2546,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);
@ -2563,8 +2557,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);
@ -2574,8 +2566,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);
@ -2585,8 +2575,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);
@ -2596,8 +2584,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);