From 2d5d2c5666cd95070f17614219c3fabe9f20a370 Mon Sep 17 00:00:00 2001 From: Tor Lillqvist Date: Tue, 2 Jan 2018 12:21:48 +0200 Subject: tdf#107039: Avoid deadlock by locking the SolarMutex early on in one place See bug report for backtraces of the deadlocking threads. Change-Id: I0334118e173c37b6f9596249e1e2f0d196bbab3f Reviewed-on: https://gerrit.libreoffice.org/47249 Tested-by: Jenkins Reviewed-by: Tor Lillqvist (cherry picked from commit de84816c1acefe0607827418f73477ff7163728d) Reviewed-on: https://gerrit.libreoffice.org/47613 Reviewed-by: Miklos Vajna --- framework/source/services/autorecovery.cxx | 1 + 1 file changed, 1 insertion(+) diff --git a/framework/source/services/autorecovery.cxx b/framework/source/services/autorecovery.cxx index 0309f91bc9da..9b56f0fb389e 100644 --- a/framework/source/services/autorecovery.cxx +++ b/framework/source/services/autorecovery.cxx @@ -1565,6 +1565,7 @@ void SAL_CALL AutoRecovery::documentEventOccured(const css::document::DocumentEv (aEvent.EventName == EVENT_ON_SAVEASDONE) ) { + SolarMutexGuard g; implts_markDocumentAsSaved(xDocument); implts_updateDocumentUsedForSavingState(xDocument, SAVE_FINISHED); } -- cgit