test: Remember to swap the window argument to XIQueryPointer

Before 5c69cb60 this wouldn't matter, because ProcXIQueryPointer
manually emitted its own error before (bogusly) returning Success to the
main loop. Since these tests only look at the return value of the
dispatch function we'd think things succeeded even when we'd generated
an error.

With that fixed, the test code's failure to swap the window id would
make dixLookupWindow (rightly) throw BadWindow.

Reviewed-and-Tested-by: Michel Dänzer <michel.daenzer@amd.com>
Signed-off-by: Adam Jackson <ajax@redhat.com>
This commit is contained in:
Adam Jackson 2016-08-17 10:32:28 -04:00
parent 5c69cb604a
commit 6acd0d098a

View File

@ -140,6 +140,7 @@ request_XIQueryPointer(ClientPtr client, xXIQueryPointerReq * req, int error)
client_request.swapped = TRUE;
swaps(&req->deviceid);
swapl(&req->win);
swaps(&req->length);
rc = SProcXIQueryPointer(&client_request);
assert(rc == error);