From f9e6858d5c10be6a8439c0f18bfb2325fa0ee070 Mon Sep 17 00:00:00 2001 From: Chase Douglas Date: Wed, 11 Jan 2012 07:38:12 -0800 Subject: Use event time instead of CurrentTime for grab times When {XI,X,}AllowEvents is called, the timestamp is compared against the grab time to ensure that the request pertains to the current grab in the server. While many clients may use CurrentTime (client-side), the timestamp of the event causing the grab is also valid. This change ensures that the server's notion of the grab time is the time of the event that activated the grab rather than the time that the grab is actually activated. This bug was exposed through nested touch then pointer grabs. Signed-off-by: Chase Douglas Reviewed-by: Peter Hutterer Signed-off-by: Peter Hutterer --- dix/events.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/dix/events.c b/dix/events.c index 0c4255cec..04d7fb59b 100644 --- a/dix/events.c +++ b/dix/events.c @@ -3723,7 +3723,8 @@ ActivatePassiveGrab(DeviceIntPtr device, GrabPtr grab, InternalEvent *event, } } - (*grabinfo->ActivateGrab)(device, grab, currentTime, TRUE); + (*grabinfo->ActivateGrab)(device, grab, + ClientTimeToServerTime(event->any.time), TRUE); if (xE) { -- cgit v1.2.3