diff options
author | Keith Packard <keithp@keithp.com> | 2017-05-10 21:50:45 -0700 |
---|---|---|
committer | Keith Packard <keithp@keithp.com> | 2017-05-12 00:21:16 -0700 |
commit | d9e23ea4228575344e3b4c0443cecc5eb75356e4 (patch) | |
tree | 70aa69f30342ba7d01f166330daca6a5bb05f021 /dix | |
parent | c394f6b27329d212d2a8b9b3c6da1ef7847c46f3 (diff) |
dix: Remove clients from input and output ready queues after closing
Delay removing the client from these two queues until all potential
I/O has completed in case we mark the client as ready for reading or
with pending output during the close operation.
Bugzilla: https://bugs.freedesktop.org/100957
Signed-off-by: Keith Packard <keithp@keithp.com>
Tested-by: Nick Sarnie <commendsarnex@gmail.com>
Reviewed-by: Michel Dänzer <michel.daenzer@amd.com>
Diffstat (limited to 'dix')
-rw-r--r-- | dix/dispatch.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/dix/dispatch.c b/dix/dispatch.c index a2df1e0e6..8b371b678 100644 --- a/dix/dispatch.c +++ b/dix/dispatch.c @@ -3414,7 +3414,6 @@ CloseDownClient(ClientPtr client) if (grabState != GrabNone && grabClient == client) { UngrabServer(client); } - mark_client_not_ready(client); BITCLEAR(grabWaiters, client->index); DeleteClientFromAnySelections(client); ReleaseActiveGrabs(client); @@ -3443,8 +3442,9 @@ CloseDownClient(ClientPtr client) if (ClientIsAsleep(client)) ClientSignal(client); ProcessWorkQueueZombies(); - output_pending_clear(client); CloseDownConnection(client); + output_pending_clear(client); + mark_client_not_ready(client); /* If the client made it to the Running stage, nClients has * been incremented on its behalf, so we need to decrement it |