summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAdam Jackson <ajax@redhat.com>2016-08-17 10:32:28 -0400
committerAdam Jackson <ajax@redhat.com>2016-08-17 11:04:13 -0400
commit6acd0d098add5a14f23d326ac6ee8cebab14b86a (patch)
tree23f4617fd773541eada34b3875daab3cc1ffb638
parent5c69cb604a144bdced7d066eaca574aa44e8cb36 (diff)
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>
-rw-r--r--test/xi2/protocol-xiquerypointer.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/test/xi2/protocol-xiquerypointer.c b/test/xi2/protocol-xiquerypointer.c
index c0421f6dd..2bf18fadc 100644
--- a/test/xi2/protocol-xiquerypointer.c
+++ b/test/xi2/protocol-xiquerypointer.c
@@ -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);