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 /Xext/saver.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 'Xext/saver.c')
-rw-r--r-- | Xext/saver.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/Xext/saver.c b/Xext/saver.c index 9e91b717b..142758c87 100644 --- a/Xext/saver.c +++ b/Xext/saver.c @@ -1443,7 +1443,6 @@ SProcScreenSaverSuspend (ClientPtr client) swaps(&stuff->length); REQUEST_SIZE_MATCH(xScreenSaverSuspendReq); - swapl(&stuff->suspend); return ProcScreenSaverSuspend (client); } |