[Libreoffice-commits] core.git: desktop/source

[Xisco Fauli (via logerrit)]

 desktop/source/deployment/gui/dp_gui_theextmgr.cxx |    6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

New commits:
commit 578758835e700b38b167753ccda9527f3a8cc43b
Author:     Xisco Fauli <xiscofa...@libreoffice.org>
AuthorDate: Thu May 25 11:15:37 2023 +0200
Commit:     Caolán McNamara <caolan.mcnam...@collabora.com>
CommitDate: Thu May 25 16:57:51 2023 +0200

    desktop: check getDialog()
    
    it might return nullptr
    
    See 
https://crashreport.libreoffice.org/stats/signature/dp_gui::TheExtensionManager::ToTop()
    
    Change-Id: I4023529aec0ba2c72b4e7db5449d235c208ee30b
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/152268
    Tested-by: Jenkins
    Tested-by: Caolán McNamara <caolan.mcnam...@collabora.com>
    Reviewed-by: Caolán McNamara <caolan.mcnam...@collabora.com>

diff --git a/desktop/source/deployment/gui/dp_gui_theextmgr.cxx 
b/desktop/source/deployment/gui/dp_gui_theextmgr.cxx
index 10a2cfb1f136..e844f09c1b85 100644
--- a/desktop/source/deployment/gui/dp_gui_theextmgr.cxx
+++ b/desktop/source/deployment/gui/dp_gui_theextmgr.cxx
@@ -158,7 +158,8 @@ void TheExtensionManager::SetText( const OUString &rTitle )
 {
     const SolarMutexGuard guard;
 
-    getDialog()->set_title( rTitle );
+    if (weld::Window* pDialog = getDialog())
+        pDialog->set_title( rTitle );
 }
 
 
@@ -166,7 +167,8 @@ void TheExtensionManager::ToTop()
 {
     const SolarMutexGuard guard;
 
-    getDialog()->present();
+    if (weld::Window* pDialog = getDialog())
+        pDialog->present();
 }
 
 void TheExtensionManager::Close()