Commit 93a050c3 authored by Tatu Frisk's avatar Tatu Frisk Committed by Adam Jackson
Browse files

Fix hanging issue in _XReply



Assume event queue is empty if another thread is blocking waiting for event.

If one thread was blocking waiting for an event and another thread sent a
reply to the X server, both threads got blocked until an event was
received.
Signed-off-by: default avatarTatu Frisk <tatu.frisk@ge.com>
Signed-off-by: default avatarJose Alarcon <jose.alarcon@ge.com>
parent 83822530
Pipeline #458338 passed with stages
in 11 minutes and 30 seconds
......@@ -704,18 +704,14 @@ Status _XReply(Display *dpy, xReply *rep, int extra, Bool discard)
if(dpy->xcb->event_owner == XlibOwnsEventQueue)
{
xcb_generic_reply_t *event;
/* If some thread is already waiting for events,
* it will get the first one. That thread must
* process that event before we can continue. */
/* FIXME: That event might be after this reply,
* and might never even come--or there might be
* multiple threads trying to get events. */
while(dpy->xcb->event_waiter)
{ /* need braces around ConditionWait */
ConditionWait(dpy, dpy->xcb->event_notify);
}
while((event = poll_for_event(dpy, True)))
handle_response(dpy, event, True);
/* Assume event queue is empty if another thread is blocking
* waiting for event. */
if(!dpy->xcb->event_waiter)
{
while((event = poll_for_response(dpy)))
handle_response(dpy, event, True);
}
}
req->reply_waiter = 0;
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment