loolwsd/LOOLKit.cpp | 2 -- 1 file changed, 2 deletions(-) New commits: commit 2a65f2e397d2b830ba02df72759883f1dbda0448 Author: Ashod Nakashian <ashod.nakash...@collabora.co.uk> Date: Sun Apr 10 13:07:20 2016 -0400
loolwsd: invoking disconnect on the session from Connection deadlocks Connection thread should not attempt to disconnect the session, which in turn will try to unload the document, which will wait on the connection to destroy. The latter will never happen since the connection destructor must, correctly, wait for its thread to finish, which is waiting on itself now. Since the session disconnect is already called from the session destructor, there is no need to explicitly invoke it here. Change-Id: Iaf9e8a10d4caa9001208084e909a14b4d4c5105e Reviewed-on: https://gerrit.libreoffice.org/23966 Reviewed-by: Ashod Nakashian <ashnak...@gmail.com> Tested-by: Ashod Nakashian <ashnak...@gmail.com> diff --git a/loolwsd/LOOLKit.cpp b/loolwsd/LOOLKit.cpp index 52083d4..ef16180 100644 --- a/loolwsd/LOOLKit.cpp +++ b/loolwsd/LOOLKit.cpp @@ -340,8 +340,6 @@ public: queue->clear(); queue->put("eof"); queueHandlerThread.join(); - - _session->disconnect(); } catch (const Exception& exc) { _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits