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/xiquerydevice.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/xiquerydevice.c')
| -rw-r--r-- | Xi/xiquerydevice.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Xi/xiquerydevice.c b/Xi/xiquerydevice.c index f5fca0d69..902eb918c 100644 --- a/Xi/xiquerydevice.c +++ b/Xi/xiquerydevice.c @@ -281,7 +281,7 @@ SwapButtonInfo(DeviceIntPtr dev, xXIButtonInfo* info) swaps(&info->sourceid); for (i = 0, btn = (Atom*)&info[1]; i < info->num_buttons; i++, btn++) - swaps(btn); + swapl(btn); swaps(&info->num_buttons); } |