(submit/fixup-req-len) xfixes: 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:34:17 +02:00
parent a93296c87e
commit ed9332b7c8
6 changed files with 0 additions and 68 deletions

View File

@ -301,8 +301,6 @@ SProcXFixesSelectCursorInput(ClientPtr client)
{ {
REQUEST(xXFixesSelectCursorInputReq); REQUEST(xXFixesSelectCursorInputReq);
REQUEST_SIZE_MATCH(xXFixesSelectCursorInputReq); REQUEST_SIZE_MATCH(xXFixesSelectCursorInputReq);
swaps(&stuff->length);
swapl(&stuff->window); swapl(&stuff->window);
swapl(&stuff->eventMask); swapl(&stuff->eventMask);
return ProcXFixesSelectCursorInput(client); return ProcXFixesSelectCursorInput(client);
@ -420,8 +418,6 @@ ProcXFixesGetCursorImage(ClientPtr client)
int _X_COLD int _X_COLD
SProcXFixesGetCursorImage(ClientPtr client) SProcXFixesGetCursorImage(ClientPtr client)
{ {
REQUEST(xXFixesGetCursorImageReq);
swaps(&stuff->length);
return ProcXFixesGetCursorImage(client); return ProcXFixesGetCursorImage(client);
} }
@ -449,8 +445,6 @@ int _X_COLD
SProcXFixesSetCursorName(ClientPtr client) SProcXFixesSetCursorName(ClientPtr client)
{ {
REQUEST(xXFixesSetCursorNameReq); REQUEST(xXFixesSetCursorNameReq);
swaps(&stuff->length);
REQUEST_AT_LEAST_SIZE(xXFixesSetCursorNameReq); REQUEST_AT_LEAST_SIZE(xXFixesSetCursorNameReq);
swapl(&stuff->cursor); swapl(&stuff->cursor);
swaps(&stuff->nbytes); swaps(&stuff->nbytes);
@ -497,8 +491,6 @@ int _X_COLD
SProcXFixesGetCursorName(ClientPtr client) SProcXFixesGetCursorName(ClientPtr client)
{ {
REQUEST(xXFixesGetCursorNameReq); REQUEST(xXFixesGetCursorNameReq);
swaps(&stuff->length);
REQUEST_SIZE_MATCH(xXFixesGetCursorNameReq); REQUEST_SIZE_MATCH(xXFixesGetCursorNameReq);
swapl(&stuff->cursor); swapl(&stuff->cursor);
return ProcXFixesGetCursorName(client); return ProcXFixesGetCursorName(client);
@ -576,8 +568,6 @@ ProcXFixesGetCursorImageAndName(ClientPtr client)
int _X_COLD int _X_COLD
SProcXFixesGetCursorImageAndName(ClientPtr client) SProcXFixesGetCursorImageAndName(ClientPtr client)
{ {
REQUEST(xXFixesGetCursorImageAndNameReq);
swaps(&stuff->length);
return ProcXFixesGetCursorImageAndName(client); return ProcXFixesGetCursorImageAndName(client);
} }
@ -704,8 +694,6 @@ int _X_COLD
SProcXFixesChangeCursor(ClientPtr client) SProcXFixesChangeCursor(ClientPtr client)
{ {
REQUEST(xXFixesChangeCursorReq); REQUEST(xXFixesChangeCursorReq);
swaps(&stuff->length);
REQUEST_SIZE_MATCH(xXFixesChangeCursorReq); REQUEST_SIZE_MATCH(xXFixesChangeCursorReq);
swapl(&stuff->source); swapl(&stuff->source);
swapl(&stuff->destination); swapl(&stuff->destination);
@ -743,8 +731,6 @@ int _X_COLD
SProcXFixesChangeCursorByName(ClientPtr client) SProcXFixesChangeCursorByName(ClientPtr client)
{ {
REQUEST(xXFixesChangeCursorByNameReq); REQUEST(xXFixesChangeCursorByNameReq);
swaps(&stuff->length);
REQUEST_AT_LEAST_SIZE(xXFixesChangeCursorByNameReq); REQUEST_AT_LEAST_SIZE(xXFixesChangeCursorByNameReq);
swapl(&stuff->source); swapl(&stuff->source);
swaps(&stuff->nbytes); swaps(&stuff->nbytes);
@ -902,8 +888,6 @@ int _X_COLD
SProcXFixesHideCursor(ClientPtr client) SProcXFixesHideCursor(ClientPtr client)
{ {
REQUEST(xXFixesHideCursorReq); REQUEST(xXFixesHideCursorReq);
swaps(&stuff->length);
REQUEST_SIZE_MATCH(xXFixesHideCursorReq); REQUEST_SIZE_MATCH(xXFixesHideCursorReq);
swapl(&stuff->window); swapl(&stuff->window);
return ProcXFixesHideCursor(client); return ProcXFixesHideCursor(client);
@ -952,8 +936,6 @@ int _X_COLD
SProcXFixesShowCursor(ClientPtr client) SProcXFixesShowCursor(ClientPtr client)
{ {
REQUEST(xXFixesShowCursorReq); REQUEST(xXFixesShowCursorReq);
swaps(&stuff->length);
REQUEST_SIZE_MATCH(xXFixesShowCursorReq); REQUEST_SIZE_MATCH(xXFixesShowCursorReq);
swapl(&stuff->window); swapl(&stuff->window);
return ProcXFixesShowCursor(client); return ProcXFixesShowCursor(client);
@ -1027,7 +1009,6 @@ SProcXFixesCreatePointerBarrier(ClientPtr client)
REQUEST_AT_LEAST_SIZE(xXFixesCreatePointerBarrierReq); REQUEST_AT_LEAST_SIZE(xXFixesCreatePointerBarrierReq);
swaps(&stuff->length);
swaps(&stuff->num_devices); swaps(&stuff->num_devices);
REQUEST_FIXED_SIZE(xXFixesCreatePointerBarrierReq, REQUEST_FIXED_SIZE(xXFixesCreatePointerBarrierReq,
pad_to_int32(stuff->num_devices * sizeof(CARD16))); pad_to_int32(stuff->num_devices * sizeof(CARD16)));
@ -1060,8 +1041,6 @@ int _X_COLD
SProcXFixesDestroyPointerBarrier(ClientPtr client) SProcXFixesDestroyPointerBarrier(ClientPtr client)
{ {
REQUEST(xXFixesDestroyPointerBarrierReq); REQUEST(xXFixesDestroyPointerBarrierReq);
swaps(&stuff->length);
REQUEST_SIZE_MATCH(xXFixesDestroyPointerBarrierReq); REQUEST_SIZE_MATCH(xXFixesDestroyPointerBarrierReq);
swapl(&stuff->barrier); swapl(&stuff->barrier);
return ProcXFixesDestroyPointerBarrier(client); return ProcXFixesDestroyPointerBarrier(client);

View File

@ -78,9 +78,6 @@ int _X_COLD
SProcXFixesSetClientDisconnectMode(ClientPtr client) SProcXFixesSetClientDisconnectMode(ClientPtr client)
{ {
REQUEST(xXFixesSetClientDisconnectModeReq); REQUEST(xXFixesSetClientDisconnectModeReq);
swaps(&stuff->length);
REQUEST_AT_LEAST_SIZE(xXFixesSetClientDisconnectModeReq); REQUEST_AT_LEAST_SIZE(xXFixesSetClientDisconnectModeReq);
swapl(&stuff->disconnect_mode); swapl(&stuff->disconnect_mode);
@ -113,12 +110,6 @@ ProcXFixesGetClientDisconnectMode(ClientPtr client)
int _X_COLD int _X_COLD
SProcXFixesGetClientDisconnectMode(ClientPtr client) SProcXFixesGetClientDisconnectMode(ClientPtr client)
{ {
REQUEST(xXFixesGetClientDisconnectModeReq);
swaps(&stuff->length);
REQUEST_SIZE_MATCH(xXFixesGetClientDisconnectModeReq);
return ProcXFixesGetClientDisconnectMode(client); return ProcXFixesGetClientDisconnectMode(client);
} }

View File

@ -97,8 +97,6 @@ int _X_COLD
SProcXFixesCreateRegion(ClientPtr client) SProcXFixesCreateRegion(ClientPtr client)
{ {
REQUEST(xXFixesCreateRegionReq); REQUEST(xXFixesCreateRegionReq);
swaps(&stuff->length);
REQUEST_AT_LEAST_SIZE(xXFixesCreateRegionReq); REQUEST_AT_LEAST_SIZE(xXFixesCreateRegionReq);
swapl(&stuff->region); swapl(&stuff->region);
SwapRestS(stuff); SwapRestS(stuff);
@ -141,8 +139,6 @@ int _X_COLD
SProcXFixesCreateRegionFromBitmap(ClientPtr client) SProcXFixesCreateRegionFromBitmap(ClientPtr client)
{ {
REQUEST(xXFixesCreateRegionFromBitmapReq); REQUEST(xXFixesCreateRegionFromBitmapReq);
swaps(&stuff->length);
REQUEST_SIZE_MATCH(xXFixesCreateRegionFromBitmapReq); REQUEST_SIZE_MATCH(xXFixesCreateRegionFromBitmapReq);
swapl(&stuff->region); swapl(&stuff->region);
swapl(&stuff->bitmap); swapl(&stuff->bitmap);
@ -200,8 +196,6 @@ int _X_COLD
SProcXFixesCreateRegionFromWindow(ClientPtr client) SProcXFixesCreateRegionFromWindow(ClientPtr client)
{ {
REQUEST(xXFixesCreateRegionFromWindowReq); REQUEST(xXFixesCreateRegionFromWindowReq);
swaps(&stuff->length);
REQUEST_SIZE_MATCH(xXFixesCreateRegionFromWindowReq); REQUEST_SIZE_MATCH(xXFixesCreateRegionFromWindowReq);
swapl(&stuff->region); swapl(&stuff->region);
swapl(&stuff->window); swapl(&stuff->window);
@ -243,8 +237,6 @@ int _X_COLD
SProcXFixesCreateRegionFromGC(ClientPtr client) SProcXFixesCreateRegionFromGC(ClientPtr client)
{ {
REQUEST(xXFixesCreateRegionFromGCReq); REQUEST(xXFixesCreateRegionFromGCReq);
swaps(&stuff->length);
REQUEST_SIZE_MATCH(xXFixesCreateRegionFromGCReq); REQUEST_SIZE_MATCH(xXFixesCreateRegionFromGCReq);
swapl(&stuff->region); swapl(&stuff->region);
swapl(&stuff->gc); swapl(&stuff->gc);
@ -285,8 +277,6 @@ int _X_COLD
SProcXFixesCreateRegionFromPicture(ClientPtr client) SProcXFixesCreateRegionFromPicture(ClientPtr client)
{ {
REQUEST(xXFixesCreateRegionFromPictureReq); REQUEST(xXFixesCreateRegionFromPictureReq);
swaps(&stuff->length);
REQUEST_SIZE_MATCH(xXFixesCreateRegionFromPictureReq); REQUEST_SIZE_MATCH(xXFixesCreateRegionFromPictureReq);
swapl(&stuff->region); swapl(&stuff->region);
swapl(&stuff->picture); swapl(&stuff->picture);
@ -309,8 +299,6 @@ int _X_COLD
SProcXFixesDestroyRegion(ClientPtr client) SProcXFixesDestroyRegion(ClientPtr client)
{ {
REQUEST(xXFixesDestroyRegionReq); REQUEST(xXFixesDestroyRegionReq);
swaps(&stuff->length);
REQUEST_SIZE_MATCH(xXFixesDestroyRegionReq); REQUEST_SIZE_MATCH(xXFixesDestroyRegionReq);
swapl(&stuff->region); swapl(&stuff->region);
return ProcXFixesDestroyRegion(client); return ProcXFixesDestroyRegion(client);
@ -347,8 +335,6 @@ int _X_COLD
SProcXFixesSetRegion(ClientPtr client) SProcXFixesSetRegion(ClientPtr client)
{ {
REQUEST(xXFixesSetRegionReq); REQUEST(xXFixesSetRegionReq);
swaps(&stuff->length);
REQUEST_AT_LEAST_SIZE(xXFixesSetRegionReq); REQUEST_AT_LEAST_SIZE(xXFixesSetRegionReq);
swapl(&stuff->region); swapl(&stuff->region);
SwapRestS(stuff); SwapRestS(stuff);
@ -376,8 +362,6 @@ int _X_COLD
SProcXFixesCopyRegion(ClientPtr client) SProcXFixesCopyRegion(ClientPtr client)
{ {
REQUEST(xXFixesCopyRegionReq); REQUEST(xXFixesCopyRegionReq);
swaps(&stuff->length);
REQUEST_SIZE_MATCH(xXFixesCopyRegionReq); REQUEST_SIZE_MATCH(xXFixesCopyRegionReq);
swapl(&stuff->source); swapl(&stuff->source);
swapl(&stuff->destination); swapl(&stuff->destination);
@ -418,8 +402,6 @@ int _X_COLD
SProcXFixesCombineRegion(ClientPtr client) SProcXFixesCombineRegion(ClientPtr client)
{ {
REQUEST(xXFixesCombineRegionReq); REQUEST(xXFixesCombineRegionReq);
swaps(&stuff->length);
REQUEST_SIZE_MATCH(xXFixesCombineRegionReq); REQUEST_SIZE_MATCH(xXFixesCombineRegionReq);
swapl(&stuff->source1); swapl(&stuff->source1);
swapl(&stuff->source2); swapl(&stuff->source2);
@ -462,8 +444,6 @@ int _X_COLD
SProcXFixesInvertRegion(ClientPtr client) SProcXFixesInvertRegion(ClientPtr client)
{ {
REQUEST(xXFixesInvertRegionReq); REQUEST(xXFixesInvertRegionReq);
swaps(&stuff->length);
REQUEST_SIZE_MATCH(xXFixesInvertRegionReq); REQUEST_SIZE_MATCH(xXFixesInvertRegionReq);
swapl(&stuff->source); swapl(&stuff->source);
swaps(&stuff->x); swaps(&stuff->x);
@ -492,8 +472,6 @@ int _X_COLD
SProcXFixesTranslateRegion(ClientPtr client) SProcXFixesTranslateRegion(ClientPtr client)
{ {
REQUEST(xXFixesTranslateRegionReq); REQUEST(xXFixesTranslateRegionReq);
swaps(&stuff->length);
REQUEST_SIZE_MATCH(xXFixesTranslateRegionReq); REQUEST_SIZE_MATCH(xXFixesTranslateRegionReq);
swapl(&stuff->region); swapl(&stuff->region);
swaps(&stuff->dx); swaps(&stuff->dx);
@ -521,8 +499,6 @@ int _X_COLD
SProcXFixesRegionExtents(ClientPtr client) SProcXFixesRegionExtents(ClientPtr client)
{ {
REQUEST(xXFixesRegionExtentsReq); REQUEST(xXFixesRegionExtentsReq);
swaps(&stuff->length);
REQUEST_SIZE_MATCH(xXFixesRegionExtentsReq); REQUEST_SIZE_MATCH(xXFixesRegionExtentsReq);
swapl(&stuff->source); swapl(&stuff->source);
swapl(&stuff->destination); swapl(&stuff->destination);
@ -586,8 +562,6 @@ int _X_COLD
SProcXFixesFetchRegion(ClientPtr client) SProcXFixesFetchRegion(ClientPtr client)
{ {
REQUEST(xXFixesFetchRegionReq); REQUEST(xXFixesFetchRegionReq);
swaps(&stuff->length);
REQUEST_SIZE_MATCH(xXFixesFetchRegionReq); REQUEST_SIZE_MATCH(xXFixesFetchRegionReq);
swapl(&stuff->region); swapl(&stuff->region);
return ProcXFixesFetchRegion(client); return ProcXFixesFetchRegion(client);
@ -645,8 +619,6 @@ int _X_COLD
SProcXFixesSetGCClipRegion(ClientPtr client) SProcXFixesSetGCClipRegion(ClientPtr client)
{ {
REQUEST(xXFixesSetGCClipRegionReq); REQUEST(xXFixesSetGCClipRegionReq);
swaps(&stuff->length);
REQUEST_SIZE_MATCH(xXFixesSetGCClipRegionReq); REQUEST_SIZE_MATCH(xXFixesSetGCClipRegionReq);
swapl(&stuff->gc); swapl(&stuff->gc);
swapl(&stuff->region); swapl(&stuff->region);
@ -748,8 +720,6 @@ int _X_COLD
SProcXFixesSetWindowShapeRegion(ClientPtr client) SProcXFixesSetWindowShapeRegion(ClientPtr client)
{ {
REQUEST(xXFixesSetWindowShapeRegionReq); REQUEST(xXFixesSetWindowShapeRegionReq);
swaps(&stuff->length);
REQUEST_SIZE_MATCH(xXFixesSetWindowShapeRegionReq); REQUEST_SIZE_MATCH(xXFixesSetWindowShapeRegionReq);
swapl(&stuff->dest); swapl(&stuff->dest);
swaps(&stuff->xOff); swaps(&stuff->xOff);
@ -797,8 +767,6 @@ int _X_COLD
SProcXFixesSetPictureClipRegion(ClientPtr client) SProcXFixesSetPictureClipRegion(ClientPtr client)
{ {
REQUEST(xXFixesSetPictureClipRegionReq); REQUEST(xXFixesSetPictureClipRegionReq);
swaps(&stuff->length);
REQUEST_SIZE_MATCH(xXFixesSetPictureClipRegionReq); REQUEST_SIZE_MATCH(xXFixesSetPictureClipRegionReq);
swapl(&stuff->picture); swapl(&stuff->picture);
swapl(&stuff->region); swapl(&stuff->region);
@ -850,8 +818,6 @@ int _X_COLD
SProcXFixesExpandRegion(ClientPtr client) SProcXFixesExpandRegion(ClientPtr client)
{ {
REQUEST(xXFixesExpandRegionReq); REQUEST(xXFixesExpandRegionReq);
swaps(&stuff->length);
REQUEST_SIZE_MATCH(xXFixesExpandRegionReq); REQUEST_SIZE_MATCH(xXFixesExpandRegionReq);
swapl(&stuff->source); swapl(&stuff->source);
swapl(&stuff->destination); swapl(&stuff->destination);

View File

@ -66,7 +66,6 @@ SProcXFixesChangeSaveSet(ClientPtr client)
REQUEST(xXFixesChangeSaveSetReq); REQUEST(xXFixesChangeSaveSetReq);
REQUEST_SIZE_MATCH(xXFixesChangeSaveSetReq); REQUEST_SIZE_MATCH(xXFixesChangeSaveSetReq);
swaps(&stuff->length);
swapl(&stuff->window); swapl(&stuff->window);
return ProcXFixesChangeSaveSet(client); return ProcXFixesChangeSaveSet(client);
} }

View File

@ -204,9 +204,7 @@ int _X_COLD
SProcXFixesSelectSelectionInput(ClientPtr client) SProcXFixesSelectSelectionInput(ClientPtr client)
{ {
REQUEST(xXFixesSelectSelectionInputReq); REQUEST(xXFixesSelectSelectionInputReq);
REQUEST_SIZE_MATCH(xXFixesSelectSelectionInputReq); REQUEST_SIZE_MATCH(xXFixesSelectSelectionInputReq);
swaps(&stuff->length);
swapl(&stuff->window); swapl(&stuff->window);
swapl(&stuff->selection); swapl(&stuff->selection);
swapl(&stuff->eventMask); swapl(&stuff->eventMask);

View File

@ -209,7 +209,6 @@ SProcXFixesQueryVersion(ClientPtr client)
REQUEST(xXFixesQueryVersionReq); REQUEST(xXFixesQueryVersionReq);
REQUEST_SIZE_MATCH(xXFixesQueryVersionReq); REQUEST_SIZE_MATCH(xXFixesQueryVersionReq);
swaps(&stuff->length);
swapl(&stuff->majorVersion); swapl(&stuff->majorVersion);
swapl(&stuff->minorVersion); swapl(&stuff->minorVersion);
return ProcXFixesQueryVersion(client); return ProcXFixesQueryVersion(client);