From 9edcae78c46286baff42e74bfe26f6ae4d00fe01 Mon Sep 17 00:00:00 2001 From: Matt Turner Date: Wed, 21 Sep 2011 17:14:16 -0400 Subject: Use correct swap{l,s} (or none at all for CARD8) Swapping the wrong size was never caught because swap{l,s} are macros. It's clear in the case of Xext/xres.c, that the author believed client_major/minor to be CARD16 from looking at the code in the first hunk. v2: dmx.c fixes from Keith. Reviewed-by: Peter Harris Signed-off-by: Matt Turner --- Xi/getdctl.c | 2 -- 1 file changed, 2 deletions(-) (limited to 'Xi/getdctl.c') diff --git a/Xi/getdctl.c b/Xi/getdctl.c index 4287028db..6090b814a 100644 --- a/Xi/getdctl.c +++ b/Xi/getdctl.c @@ -127,7 +127,6 @@ static void CopySwapDeviceCore (ClientPtr client, DeviceIntPtr dev, char *buf) if (client->swapped) { swaps(&c->control); swaps(&c->length); - swaps(&c->status); } } @@ -142,7 +141,6 @@ static void CopySwapDeviceEnable (ClientPtr client, DeviceIntPtr dev, char *buf) if (client->swapped) { swaps(&e->control); swaps(&e->length); - swaps(&e->enable); } } -- cgit v1.2.3