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/xichangehierarchy.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/xichangehierarchy.c')
| -rw-r--r-- | Xi/xichangehierarchy.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Xi/xichangehierarchy.c b/Xi/xichangehierarchy.c index f2bd8bb05..614d23116 100644 --- a/Xi/xichangehierarchy.c +++ b/Xi/xichangehierarchy.c @@ -434,7 +434,7 @@ ProcXIChangeHierarchy(ClientPtr client) any = (xXIAnyHierarchyChangeInfo*)&stuff[1]; while(stuff->num_changes--) { - SWAPIF(swapl(&any->type)); + SWAPIF(swaps(&any->type)); SWAPIF(swaps(&any->length)); required_len += any->length; |