Null pointer dereferences must not happen

This is a follow-up of 09981cd6383ecb99e4b6c83b98b03af5cf3ff59b.
Inspired by coverity#1362478 etc.

Change-Id: If5507c52552393cccbb255cbf4b7e9f4b80db917
Reviewed-on: https://gerrit.libreoffice.org/25840
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>

1 files changed, 4 insertions, 1 deletions

diff --git a/starmath/source/cursor.cxx b/starmath/source/cursor.cxx
index 3baf5e1b9f78..dab3dc497974 100644
--- a/starmath/source/cursor.cxx
+++ b/starmath/source/cursor.cxx
@@ -139,11 +139,14 @@ void SmCursor::BuildGraph(){
     }
     //Set position and anchor to first caret position
     auto it = mpGraph->begin();
+    assert(it != mpGraph->end());
     if(!mpPosition)
-        mpPosition = (it == mpGraph->end()) ? nullptr : it->get();
+        mpPosition = it->get();
     if(!mpAnchor)
         mpAnchor = mpPosition;
 
+    assert(mpPosition);
+    assert(mpAnchor);
     OSL_ENSURE(mpPosition->CaretPos.IsValid(), "Position must be valid");
     OSL_ENSURE(mpAnchor->CaretPos.IsValid(), "Anchor must be valid");
 }