[Libreoffice-commits] core.git: cui/source vcl/source

[Xisco Fauli]

 cui/source/options/personalization.cxx |    4 ++++
 vcl/source/window/msgbox.cxx           |    2 ++
 2 files changed, 6 insertions(+)

New commits:
commit 0ddd15c1134dde249552ebe763c1cd005e8b4591
Author: Xisco Fauli <anistenis at gmail.com>
Date:   Thu Jan 4 12:41:07 2018 +0100

    tdf#114818 hold the mutex before executing the error dialog
    
    Also add an assert to ErrorBox to ensure the mutex is acquired
    
    Change-Id: I3e426024dad6ddfc5cffc7861b0cba9e8989c018
    Reviewed-on: https://gerrit.libreoffice.org/47388
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>

diff --git a/cui/source/options/personalization.cxx b/cui/source/options/personalization.cxx
index d18d56b06fb6..c75a78c8ef2e 100644
--- a/cui/source/options/personalization.cxx
+++ b/cui/source/options/personalization.cxx
@@ -702,6 +702,7 @@ void SearchAndParseThread::execute()
                 {
                     // in case of a returned CommandFailedException
                     // SimpleFileAccess serves it, returning an empty stream
+                    SolarMutexGuard aGuard;
                     sError = CuiResId(RID_SVXSTR_SEARCHERROR);
                     sError = sError.replaceAll("%1", m_aURL);
                     m_pPersonaDialog->SetProgress( OUString() );
@@ -714,6 +715,7 @@ void SearchAndParseThread::execute()
             {
                 // a catch all clause, in case the exception is not
                 // served elsewhere
+                SolarMutexGuard aGuard;
                 sError = CuiResId(RID_SVXSTR_SEARCHERROR);
                 sError = sError.replaceAll("%1", m_aURL);
                 m_pPersonaDialog->SetProgress( OUString() );
@@ -754,6 +756,7 @@ void SearchAndParseThread::execute()
             {
                 if( m_bDirectURL )
                 {
+                    SolarMutexGuard aGuard;
                     sError = CuiResId(RID_SVXSTR_SEARCHERROR);
                     sError = sError.replaceAll("%1", m_aURL);
                     m_pPersonaDialog->SetProgress( OUString() );
@@ -837,6 +840,7 @@ void SearchAndParseThread::execute()
         }
         catch ( const uno::Exception & )
         {
+            SolarMutexGuard aGuard;
             sError = CuiResId( RID_SVXSTR_SEARCHERROR );
             sError = sError.replaceAll("%1", m_aURL);
             m_pPersonaDialog->SetProgress( OUString() );
diff --git a/vcl/source/window/msgbox.cxx b/vcl/source/window/msgbox.cxx
index b86335840bb7..428e4f31ef08 100644
--- a/vcl/source/window/msgbox.cxx
+++ b/vcl/source/window/msgbox.cxx
@@ -433,6 +433,7 @@ OUString WarningBox::GetStandardText()
 ErrorBox::ErrorBox( vcl::Window* pParent, const OUString& rMessage ) :
     MessBox( pParent, MessBoxStyle::Ok | MessBoxStyle::DefaultOk, 0, OUString(), rMessage )
 {
+    DBG_TESTSOLARMUTEX();
     // Default Text is the display title from the application
     if ( GetText().isEmpty() )
         SetText( GetStandardText() );
@@ -450,6 +451,7 @@ ErrorBox::ErrorBox( vcl::Window* pParent, MessBoxStyle nStyle, WinBits nWinBits,
                     const OUString& rMessage ) :
     MessBox( pParent, nStyle, nWinBits, OUString(), rMessage )
 {
+    DBG_TESTSOLARMUTEX();
     // Default Text is the display title from the application
     if ( GetText().isEmpty() )
         SetText( GetStandardText() );