diff options
author | Matt Turner <mattst88@gmail.com> | 2011-09-21 17:14:16 -0400 |
---|---|---|
committer | Matt Turner <mattst88@gmail.com> | 2011-09-21 17:14:16 -0400 |
commit | 9edcae78c46286baff42e74bfe26f6ae4d00fe01 (patch) | |
tree | 7be712a39c5fdc3c7b648f7cecbd5e058535599f /Xi/getdctl.c | |
parent | 2c7c520cfe0df30f4bc3adba59d9c62582823bf8 (diff) |
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 <pharris@opentext.com>
Signed-off-by: Matt Turner <mattst88@gmail.com>
Diffstat (limited to 'Xi/getdctl.c')
-rw-r--r-- | Xi/getdctl.c | 2 |
1 files changed, 0 insertions, 2 deletions
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); } } |