diff options
author | Adam Jackson <ajax@redhat.com> | 2016-08-17 10:32:28 -0400 |
---|---|---|
committer | Adam Jackson <ajax@redhat.com> | 2016-08-17 11:04:13 -0400 |
commit | 6acd0d098add5a14f23d326ac6ee8cebab14b86a (patch) | |
tree | 23f4617fd773541eada34b3875daab3cc1ffb638 | |
parent | 5c69cb604a144bdced7d066eaca574aa44e8cb36 (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.c | 1 |
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); |