loolwsd/LOOLKit.cpp | 5 +++++ 1 file changed, 5 insertions(+) New commits: commit 2103fe2515b7c71aff546dc00fe8a46721abdfa2 Author: Tor Lillqvist <t...@collabora.com> Date: Thu Mar 3 18:35:48 2016 +0200
Revert "We use a recursive mutex, so no need to drop and re-take around documentLoad()" Turns out the callbacks from documentLoad() are in general done on another thread, not necessarily the same one that did the documentLoad() call. Not sure why it happened to be the same thread for me, but oh well. Need to fix the problem described in 3671abf89b376555394b22bfee25d450e7eff5f6 in some other way then. This reverts commit 2fab7574626919d66c637f40302cb00e8d280b3f. diff --git a/loolwsd/LOOLKit.cpp b/loolwsd/LOOLKit.cpp index 7942436..31b659d 100644 --- a/loolwsd/LOOLKit.cpp +++ b/loolwsd/LOOLKit.cpp @@ -620,6 +620,9 @@ private: LOK_FEATURE_DOCUMENT_PASSWORD_TO_MODIFY); } + // documentLoad will trigger callback, which needs to take the lock. + lock.unlock(); + // Save the provided password with us and the jailed url _isDocPasswordProvided = isDocPasswordProvided; _docPassword = docPassword; @@ -649,6 +652,8 @@ private: return nullptr; } Log::info("documentLoad() returned"); + // Retake the lock. + lock.lock(); if (_multiView) { _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits