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/xichangehierarchy.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'Xi/xichangehierarchy.c') 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; -- cgit v1.2.3