assure pointer and position in OutlineContentVisibilityWin Set

won't cause crash.

Change-Id: Ic1bb2a1754c1a5a837370c7bb91bf7356e14b6bb
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100716
Tested-by: Jenkins
Reviewed-by: Jim Raykowski <raykowj@gmail.com>

1 files changed, 8 insertions, 3 deletions

diff --git a/sw/source/uibase/docvw/OutlineContentVisibilityWin.cxx b/sw/source/uibase/docvw/OutlineContentVisibilityWin.cxx
index ffebaf3e3f00..2bbc55504e95 100644
--- a/sw/source/uibase/docvw/OutlineContentVisibilityWin.cxx
+++ b/sw/source/uibase/docvw/OutlineContentVisibilityWin.cxx
@@ -66,9 +66,14 @@ void SwOutlineContentVisibilityWin::Set()
     const SwTextNode* pTextNode = pTextFrame->GetTextNodeFirst();
     SwWrtShell& rSh = GetEditWin()->GetView().GetWrtShell();
     const SwOutlineNodes& rOutlineNodes = rSh.GetNodes().GetOutLineNds();
-    (void)rOutlineNodes.Seek_Entry(static_cast<SwNode*>(const_cast<SwTextNode*>(pTextNode)),
-                                   &m_nOutlinePos);
-    assert(m_nOutlinePos != SwOutlineNodes::npos);
+    if (!pTextNode
+        || !rOutlineNodes.Seek_Entry(static_cast<SwNode*>(const_cast<SwTextNode*>(pTextNode)),
+                                     &m_nOutlinePos)
+        || m_nOutlinePos == SwOutlineNodes::npos)
+    {
+        assert(false); // should never get here
+        return;
+    }
 
     // don't set if no content and no subs with content
     auto nPos = m_nOutlinePos;