cui/source/dialogs/hldoctp.cxx |   11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

New commits:
commit d7235d9099e5b7db80958b4a79801581198aab08
Author:     Noel Grandin <noel.gran...@collabora.co.uk>
AuthorDate: Thu Sep 5 12:11:27 2024 +0200
Commit:     Xisco Fauli <xiscofa...@libreoffice.org>
CommitDate: Thu Sep 5 20:05:38 2024 +0200

    fix crash in Hyperlink dialog
    
    which I managed to trigger by hitting escape repeatedly while it was
    trying to load the targets window
    
    Change-Id: I4c7cb3a30104c58bfc6652923a0aa2169c1bf3bf
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/172907
    Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>
    Tested-by: Jenkins
    (cherry picked from commit 6633afe0ff18e5921eb96be2f863f28bd7e1110d)
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/172894
    Reviewed-by: Xisco Fauli <xiscofa...@libreoffice.org>
    Signed-off-by: Xisco Fauli <xiscofa...@libreoffice.org>
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/172918

diff --git a/cui/source/dialogs/hldoctp.cxx b/cui/source/dialogs/hldoctp.cxx
index a1c1454b69de..8c65fdb47fd6 100644
--- a/cui/source/dialogs/hldoctp.cxx
+++ b/cui/source/dialogs/hldoctp.cxx
@@ -253,10 +253,13 @@ IMPL_LINK_NOARG(SvxHyperlinkDocTp, TimeoutHdl_Impl, Timer 
*, void)
     {
         weld::WaitObject aWait(mpDialog->getDialog());
 
-        if ( maStrURL.equalsIgnoreAsciiCase( INET_FILE_SCHEME ) )
-            mxMarkWnd->RefreshTree ( "" );
-        else
-            mxMarkWnd->RefreshTree ( maStrURL );
+        if (mxMarkWnd)
+        {
+            if ( maStrURL.equalsIgnoreAsciiCase( INET_FILE_SCHEME ) )
+                mxMarkWnd->RefreshTree ( "" );
+            else
+                mxMarkWnd->RefreshTree ( maStrURL );
+        }
     }
 }
 

Reply via email to