(submit/fixup-req-len) dix: 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.

Also dropping now obsolete SProcNoOperation().

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>

dix/swapreq.c

@@ -120,7 +120,6 @@ int _X_COLD
 SProcSimpleReq(ClientPtr client)
 {
     REQUEST(xReq);
-    swaps(&stuff->length);
     return (*ProcVector[stuff->reqType]) (client);
 }
 
@@ -131,7 +130,6 @@ int _X_COLD
 SProcResourceReq(ClientPtr client)
 {
     REQUEST(xResourceReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xResourceReq);        /* not EXACT */
     swapl(&stuff->id);
     return (*ProcVector[stuff->reqType]) (client);
@@ -141,7 +139,6 @@ int _X_COLD
 SProcCreateWindow(ClientPtr client)
 {
     REQUEST(xCreateWindowReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xCreateWindowReq);
     swapl(&stuff->wid);
     swapl(&stuff->parent);
@@ -161,7 +158,6 @@ int _X_COLD
 SProcChangeWindowAttributes(ClientPtr client)
 {
     REQUEST(xChangeWindowAttributesReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xChangeWindowAttributesReq);
     swapl(&stuff->window);
     swapl(&stuff->valueMask);
@@ -173,7 +169,6 @@ int _X_COLD
 SProcReparentWindow(ClientPtr client)
 {
     REQUEST(xReparentWindowReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xReparentWindowReq);
     swapl(&stuff->window);
     swapl(&stuff->parent);
@@ -186,7 +181,6 @@ int _X_COLD
 SProcConfigureWindow(ClientPtr client)
 {
     REQUEST(xConfigureWindowReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xConfigureWindowReq);
     swapl(&stuff->window);
     swaps(&stuff->mask);
@@ -199,7 +193,6 @@ int _X_COLD
 SProcInternAtom(ClientPtr client)
 {
     REQUEST(xInternAtomReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xInternAtomReq);
     swaps(&stuff->nbytes);
     return ((*ProcVector[X_InternAtom]) (client));
@@ -209,7 +202,6 @@ int _X_COLD
 SProcChangeProperty(ClientPtr client)
 {
     REQUEST(xChangePropertyReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xChangePropertyReq);
     swapl(&stuff->window);
     swapl(&stuff->property);
@@ -232,19 +224,16 @@ int _X_COLD
 SProcDeleteProperty(ClientPtr client)
 {
     REQUEST(xDeletePropertyReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xDeletePropertyReq);
     swapl(&stuff->window);
     swapl(&stuff->property);
     return ((*ProcVector[X_DeleteProperty]) (client));
-
 }
 
 int _X_COLD
 SProcGetProperty(ClientPtr client)
 {
     REQUEST(xGetPropertyReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xGetPropertyReq);
     swapl(&stuff->window);
     swapl(&stuff->property);
@@ -258,7 +247,6 @@ int _X_COLD
 SProcSetSelectionOwner(ClientPtr client)
 {
     REQUEST(xSetSelectionOwnerReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xSetSelectionOwnerReq);
     swapl(&stuff->window);
     swapl(&stuff->selection);
@@ -270,7 +258,6 @@ int _X_COLD
 SProcConvertSelection(ClientPtr client)
 {
     REQUEST(xConvertSelectionReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xConvertSelectionReq);
     swapl(&stuff->requestor);
     swapl(&stuff->selection);
@@ -287,7 +274,6 @@ SProcSendEvent(ClientPtr client)
     EventSwapPtr proc;
 
     REQUEST(xSendEventReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xSendEventReq);
     swapl(&stuff->destination);
     swapl(&stuff->eventMask);
@@ -313,7 +299,6 @@ int _X_COLD
 SProcGrabPointer(ClientPtr client)
 {
     REQUEST(xGrabPointerReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xGrabPointerReq);
     swapl(&stuff->grabWindow);
     swaps(&stuff->eventMask);
@@ -327,7 +312,6 @@ int _X_COLD
 SProcGrabButton(ClientPtr client)
 {
     REQUEST(xGrabButtonReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xGrabButtonReq);
     swapl(&stuff->grabWindow);
     swaps(&stuff->eventMask);
@@ -341,7 +325,6 @@ int _X_COLD
 SProcUngrabButton(ClientPtr client)
 {
     REQUEST(xUngrabButtonReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xUngrabButtonReq);
     swapl(&stuff->grabWindow);
     swaps(&stuff->modifiers);
@@ -352,7 +335,6 @@ int _X_COLD
 SProcChangeActivePointerGrab(ClientPtr client)
 {
     REQUEST(xChangeActivePointerGrabReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xChangeActivePointerGrabReq);
     swapl(&stuff->cursor);
     swapl(&stuff->time);
@@ -364,7 +346,6 @@ int _X_COLD
 SProcGrabKeyboard(ClientPtr client)
 {
     REQUEST(xGrabKeyboardReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xGrabKeyboardReq);
     swapl(&stuff->grabWindow);
     swapl(&stuff->time);
@@ -375,7 +356,6 @@ int _X_COLD
 SProcGrabKey(ClientPtr client)
 {
     REQUEST(xGrabKeyReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xGrabKeyReq);
     swapl(&stuff->grabWindow);
     swaps(&stuff->modifiers);
@@ -386,7 +366,6 @@ int _X_COLD
 SProcUngrabKey(ClientPtr client)
 {
     REQUEST(xUngrabKeyReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xUngrabKeyReq);
     swapl(&stuff->grabWindow);
     swaps(&stuff->modifiers);
@@ -397,7 +376,6 @@ int _X_COLD
 SProcGetMotionEvents(ClientPtr client)
 {
     REQUEST(xGetMotionEventsReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xGetMotionEventsReq);
     swapl(&stuff->window);
     swapl(&stuff->start);
@@ -409,7 +387,6 @@ int _X_COLD
 SProcTranslateCoords(ClientPtr client)
 {
     REQUEST(xTranslateCoordsReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xTranslateCoordsReq);
     swapl(&stuff->srcWid);
     swapl(&stuff->dstWid);
@@ -422,7 +399,6 @@ int _X_COLD
 SProcWarpPointer(ClientPtr client)
 {
     REQUEST(xWarpPointerReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xWarpPointerReq);
     swapl(&stuff->srcWid);
     swapl(&stuff->dstWid);
@@ -439,7 +415,6 @@ int _X_COLD
 SProcSetInputFocus(ClientPtr client)
 {
     REQUEST(xSetInputFocusReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xSetInputFocusReq);
     swapl(&stuff->focus);
     swapl(&stuff->time);
@@ -450,7 +425,6 @@ int _X_COLD
 SProcOpenFont(ClientPtr client)
 {
     REQUEST(xOpenFontReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xOpenFontReq);
     swapl(&stuff->fid);
     swaps(&stuff->nbytes);
@@ -461,7 +435,6 @@ int _X_COLD
 SProcListFonts(ClientPtr client)
 {
     REQUEST(xListFontsReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xListFontsReq);
     swaps(&stuff->maxNames);
     swaps(&stuff->nbytes);
@@ -472,7 +445,6 @@ int _X_COLD
 SProcListFontsWithInfo(ClientPtr client)
 {
     REQUEST(xListFontsWithInfoReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xListFontsWithInfoReq);
     swaps(&stuff->maxNames);
     swaps(&stuff->nbytes);
@@ -483,7 +455,6 @@ int _X_COLD
 SProcSetFontPath(ClientPtr client)
 {
     REQUEST(xSetFontPathReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xSetFontPathReq);
     swaps(&stuff->nFonts);
     return ((*ProcVector[X_SetFontPath]) (client));
@@ -493,8 +464,6 @@ int _X_COLD
 SProcCreatePixmap(ClientPtr client)
 {
     REQUEST(xCreatePixmapReq);
-
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xCreatePixmapReq);
     swapl(&stuff->pid);
     swapl(&stuff->drawable);
@@ -507,7 +476,6 @@ int _X_COLD
 SProcCreateGC(ClientPtr client)
 {
     REQUEST(xCreateGCReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xCreateGCReq);
     swapl(&stuff->gc);
     swapl(&stuff->drawable);
@@ -520,7 +488,6 @@ int _X_COLD
 SProcChangeGC(ClientPtr client)
 {
     REQUEST(xChangeGCReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xChangeGCReq);
     swapl(&stuff->gc);
     swapl(&stuff->mask);
@@ -532,7 +499,6 @@ int _X_COLD
 SProcCopyGC(ClientPtr client)
 {
     REQUEST(xCopyGCReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xCopyGCReq);
     swapl(&stuff->srcGC);
     swapl(&stuff->dstGC);
@@ -544,20 +510,17 @@ int _X_COLD
 SProcSetDashes(ClientPtr client)
 {
     REQUEST(xSetDashesReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xSetDashesReq);
     swapl(&stuff->gc);
     swaps(&stuff->dashOffset);
     swaps(&stuff->nDashes);
     return ((*ProcVector[X_SetDashes]) (client));
-
 }
 
 int _X_COLD
 SProcSetClipRectangles(ClientPtr client)
 {
     REQUEST(xSetClipRectanglesReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xSetClipRectanglesReq);
     swapl(&stuff->gc);
     swaps(&stuff->xOrigin);
@@ -570,7 +533,6 @@ int _X_COLD
 SProcClearToBackground(ClientPtr client)
 {
     REQUEST(xClearAreaReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xClearAreaReq);
     swapl(&stuff->window);
     swaps(&stuff->x);
@@ -584,7 +546,6 @@ int _X_COLD
 SProcCopyArea(ClientPtr client)
 {
     REQUEST(xCopyAreaReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xCopyAreaReq);
     swapl(&stuff->srcDrawable);
     swapl(&stuff->dstDrawable);
@@ -602,7 +563,6 @@ int _X_COLD
 SProcCopyPlane(ClientPtr client)
 {
     REQUEST(xCopyPlaneReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xCopyPlaneReq);
     swapl(&stuff->srcDrawable);
     swapl(&stuff->dstDrawable);
@@ -623,7 +583,6 @@ int _X_COLD
 SProcPoly(ClientPtr client)
 {
     REQUEST(xPolyPointReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xPolyPointReq);
     swapl(&stuff->drawable);
     swapl(&stuff->gc);
@@ -638,7 +597,6 @@ int _X_COLD
 SProcFillPoly(ClientPtr client)
 {
     REQUEST(xFillPolyReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xFillPolyReq);
     swapl(&stuff->drawable);
     swapl(&stuff->gc);
@@ -650,7 +608,6 @@ int _X_COLD
 SProcPutImage(ClientPtr client)
 {
     REQUEST(xPutImageReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xPutImageReq);
     swapl(&stuff->drawable);
     swapl(&stuff->gc);
@@ -660,14 +617,12 @@ SProcPutImage(ClientPtr client)
     swaps(&stuff->dstY);
     /* Image should already be swapped */
     return ((*ProcVector[X_PutImage]) (client));
-
 }
 
 int _X_COLD
 SProcGetImage(ClientPtr client)
 {
     REQUEST(xGetImageReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xGetImageReq);
     swapl(&stuff->drawable);
     swaps(&stuff->x);
@@ -684,7 +639,6 @@ int _X_COLD
 SProcPolyText(ClientPtr client)
 {
     REQUEST(xPolyTextReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xPolyTextReq);
     swapl(&stuff->drawable);
     swapl(&stuff->gc);
@@ -699,7 +653,6 @@ int _X_COLD
 SProcImageText(ClientPtr client)
 {
     REQUEST(xImageTextReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xImageTextReq);
     swapl(&stuff->drawable);
     swapl(&stuff->gc);
@@ -712,7 +665,6 @@ int _X_COLD
 SProcCreateColormap(ClientPtr client)
 {
     REQUEST(xCreateColormapReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xCreateColormapReq);
     swapl(&stuff->mid);
     swapl(&stuff->window);
@@ -724,19 +676,16 @@ int _X_COLD
 SProcCopyColormapAndFree(ClientPtr client)
 {
     REQUEST(xCopyColormapAndFreeReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xCopyColormapAndFreeReq);
     swapl(&stuff->mid);
     swapl(&stuff->srcCmap);
     return ((*ProcVector[X_CopyColormapAndFree]) (client));
-
 }
 
 int _X_COLD
 SProcAllocColor(ClientPtr client)
 {
     REQUEST(xAllocColorReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xAllocColorReq);
     swapl(&stuff->cmap);
     swaps(&stuff->red);
@@ -749,7 +698,6 @@ int _X_COLD
 SProcAllocNamedColor(ClientPtr client)
 {
     REQUEST(xAllocNamedColorReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xAllocNamedColorReq);
     swapl(&stuff->cmap);
     swaps(&stuff->nbytes);
@@ -760,7 +708,6 @@ int _X_COLD
 SProcAllocColorCells(ClientPtr client)
 {
     REQUEST(xAllocColorCellsReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xAllocColorCellsReq);
     swapl(&stuff->cmap);
     swaps(&stuff->colors);
@@ -772,7 +719,6 @@ int _X_COLD
 SProcAllocColorPlanes(ClientPtr client)
 {
     REQUEST(xAllocColorPlanesReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xAllocColorPlanesReq);
     swapl(&stuff->cmap);
     swaps(&stuff->colors);
@@ -786,13 +732,11 @@ int _X_COLD
 SProcFreeColors(ClientPtr client)
 {
     REQUEST(xFreeColorsReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xFreeColorsReq);
     swapl(&stuff->cmap);
     swapl(&stuff->planeMask);
     SwapRestL(stuff);
     return ((*ProcVector[X_FreeColors]) (client));
-
 }
 
 void _X_COLD
@@ -811,7 +755,6 @@ SProcStoreColors(ClientPtr client)
     xColorItem *pItem;
 
     REQUEST(xStoreColorsReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xStoreColorsReq);
     swapl(&stuff->cmap);
     pItem = (xColorItem *) &stuff[1];
@@ -824,7 +767,6 @@ int _X_COLD
 SProcStoreNamedColor(ClientPtr client)
 {
     REQUEST(xStoreNamedColorReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xStoreNamedColorReq);
     swapl(&stuff->cmap);
     swapl(&stuff->pixel);
@@ -836,7 +778,6 @@ int _X_COLD
 SProcQueryColors(ClientPtr client)
 {
     REQUEST(xQueryColorsReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xQueryColorsReq);
     swapl(&stuff->cmap);
     SwapRestL(stuff);
@@ -847,7 +788,6 @@ int _X_COLD
 SProcLookupColor(ClientPtr client)
 {
     REQUEST(xLookupColorReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xLookupColorReq);
     swapl(&stuff->cmap);
     swaps(&stuff->nbytes);
@@ -858,7 +798,6 @@ int _X_COLD
 SProcCreateCursor(ClientPtr client)
 {
     REQUEST(xCreateCursorReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xCreateCursorReq);
     swapl(&stuff->cid);
     swapl(&stuff->source);
@@ -878,7 +817,6 @@ int _X_COLD
 SProcCreateGlyphCursor(ClientPtr client)
 {
     REQUEST(xCreateGlyphCursorReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xCreateGlyphCursorReq);
     swapl(&stuff->cid);
     swapl(&stuff->source);
@@ -898,7 +836,6 @@ int _X_COLD
 SProcRecolorCursor(ClientPtr client)
 {
     REQUEST(xRecolorCursorReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xRecolorCursorReq);
     swapl(&stuff->cursor);
     swaps(&stuff->foreRed);
@@ -914,20 +851,17 @@ int _X_COLD
 SProcQueryBestSize(ClientPtr client)
 {
     REQUEST(xQueryBestSizeReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xQueryBestSizeReq);
     swapl(&stuff->drawable);
     swaps(&stuff->width);
     swaps(&stuff->height);
     return ((*ProcVector[X_QueryBestSize]) (client));
-
 }
 
 int _X_COLD
 SProcQueryExtension(ClientPtr client)
 {
     REQUEST(xQueryExtensionReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xQueryExtensionReq);
     swaps(&stuff->nbytes);
     return ((*ProcVector[X_QueryExtension]) (client));
@@ -937,7 +871,6 @@ int _X_COLD
 SProcChangeKeyboardMapping(ClientPtr client)
 {
     REQUEST(xChangeKeyboardMappingReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xChangeKeyboardMappingReq);
     SwapRestL(stuff);
     return ((*ProcVector[X_ChangeKeyboardMapping]) (client));
@@ -947,7 +880,6 @@ int _X_COLD
 SProcChangeKeyboardControl(ClientPtr client)
 {
     REQUEST(xChangeKeyboardControlReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xChangeKeyboardControlReq);
     swapl(&stuff->mask);
     SwapRestL(stuff);
@@ -958,7 +890,6 @@ int _X_COLD
 SProcChangePointerControl(ClientPtr client)
 {
     REQUEST(xChangePointerControlReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xChangePointerControlReq);
     swaps(&stuff->accelNum);
     swaps(&stuff->accelDenum);
@@ -970,7 +901,6 @@ int _X_COLD
 SProcSetScreenSaver(ClientPtr client)
 {
     REQUEST(xSetScreenSaverReq);
-    swaps(&stuff->length);
     REQUEST_SIZE_MATCH(xSetScreenSaverReq);
     swaps(&stuff->timeout);
     swaps(&stuff->interval);
@@ -981,18 +911,15 @@ int _X_COLD
 SProcChangeHosts(ClientPtr client)
 {
     REQUEST(xChangeHostsReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xChangeHostsReq);
     swaps(&stuff->hostLength);
     return ((*ProcVector[X_ChangeHosts]) (client));
-
 }
 
 int _X_COLD
 SProcRotateProperties(ClientPtr client)
 {
     REQUEST(xRotatePropertiesReq);
-    swaps(&stuff->length);
     REQUEST_AT_LEAST_SIZE(xRotatePropertiesReq);
     swapl(&stuff->window);
     swaps(&stuff->nAtoms);
@@ -1001,14 +928,6 @@ SProcRotateProperties(ClientPtr client)
     return ((*ProcVector[X_RotateProperties]) (client));
 }
 
-int _X_COLD
-SProcNoOperation(ClientPtr client)
-{
-    REQUEST(xReq);
-    swaps(&stuff->length);
-    return ((*ProcVector[X_NoOperation]) (client));
-}
-
 void _X_COLD
 SwapConnClientPrefix(xConnClientPrefix * pCCP)
 {

dix/tables.c

@@ -451,7 +451,7 @@ int (*SwappedProcVector[256]) (ClientPtr /* client */) = {
     ProcBadRequest,
     ProcBadRequest,                     /* 125 */
     ProcBadRequest,
-    SProcNoOperation,
+    ProcNoOperation,
     ProcBadRequest,
     ProcBadRequest,
     ProcBadRequest,

include/swapreq.h

@@ -74,7 +74,6 @@ SWAPREQ_PROC(SProcInternAtom);
 SWAPREQ_PROC(SProcListFonts);
 SWAPREQ_PROC(SProcListFontsWithInfo);
 SWAPREQ_PROC(SProcLookupColor);
-SWAPREQ_PROC(SProcNoOperation);
 SWAPREQ_PROC(SProcOpenFont);
 SWAPREQ_PROC(SProcPoly);
 SWAPREQ_PROC(SProcPolyText);