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 /randr | |
| 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 'randr')
| -rw-r--r-- | randr/rrcrtc.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/randr/rrcrtc.c b/randr/rrcrtc.c index a8b73d9b7..e6a38ae60 100644 --- a/randr/rrcrtc.c +++ b/randr/rrcrtc.c @@ -1117,7 +1117,7 @@ ProcRRGetPanning (ClientPtr client) if (client->swapped) { swaps(&rep.sequenceNumber); swapl(&rep.length); - swaps(&rep.timestamp); + swapl(&rep.timestamp); swaps(&rep.left); swaps(&rep.top); swaps(&rep.width); @@ -1197,7 +1197,7 @@ sendReply: if (client->swapped) { swaps(&rep.sequenceNumber); swapl(&rep.length); - swaps(&rep.newTimestamp); + swapl(&rep.newTimestamp); } WriteToClient(client, sizeof(xRRSetPanningReply), (char *)&rep); return Success; |