xcb_request_check: Sync even if an event was read for this sequence.
This fixes the test case I have so far for Havoc's report that xcb_request_check hangs. Rationale: Since we have a void cookie, request_expected can't have been set equal to this sequence number when the request was sent; it can only have become equal due to the arrival of an event or error. If it became equal due to an event then we still need to sync. If it became equal due to an error, then request_completed will have been updated, which means we correctly won't sync. Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=29599 However, Havoc reports that he can still reproduce the problem, so we may be revisiting this later. Reported-by: Havoc Pennington <hp@pobox.com> Signed-off-by: Jamey Sharp <jamey@minilop.net>
This commit is contained in:
parent
35f901a0f2
commit
2040f10a4e
|
@ -564,7 +564,7 @@ xcb_generic_error_t *xcb_request_check(xcb_connection_t *c, xcb_void_cookie_t co
|
||||||
void *reply;
|
void *reply;
|
||||||
if(c->has_error)
|
if(c->has_error)
|
||||||
return 0;
|
return 0;
|
||||||
if(XCB_SEQUENCE_COMPARE_32(cookie.sequence,>,c->in.request_expected)
|
if(XCB_SEQUENCE_COMPARE_32(cookie.sequence,>=,c->in.request_expected)
|
||||||
&& XCB_SEQUENCE_COMPARE_32(cookie.sequence,>,c->in.request_completed))
|
&& XCB_SEQUENCE_COMPARE_32(cookie.sequence,>,c->in.request_completed))
|
||||||
{
|
{
|
||||||
free(xcb_get_input_focus_reply(c, xcb_get_input_focus(c), &ret));
|
free(xcb_get_input_focus_reply(c, xcb_get_input_focus(c), &ret));
|
||||||
|
|
Loading…
Reference in New Issue