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

Caolán McNamara caolanm at redhat.com
Tue Nov 5 17:13:36 CET 2013


 starmath/source/mathmlimport.cxx |    5 +++--
 starmath/source/node.cxx         |    3 ++-
 2 files changed, 5 insertions(+), 3 deletions(-)

New commits:
commit 603e9338568c29c01f812c4fd47b7e8138e015bb
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Tue Nov 5 16:04:28 2013 +0000

    Resolves: fdo#58094 don't crash on attempting mml import
    
    Change-Id: I812f2b3e333ea59e9ebfc75a854faca050af8711

diff --git a/starmath/source/mathmlimport.cxx b/starmath/source/mathmlimport.cxx
index 073d96c..f8a272e 100644
--- a/starmath/source/mathmlimport.cxx
+++ b/starmath/source/mathmlimport.cxx
@@ -2248,10 +2248,11 @@ void SmXMLRowContext_Impl::EndElement()
 {
     SmNodeArray aRelationArray;
     SmNodeStack &rNodeStack = GetSmImport().GetNodeStack();
-    sal_uLong nSize = rNodeStack.size()-nElementCount;
 
-    if (nSize > 0)
+    if (rNodeStack.size() > nElementCount)
     {
+        sal_uLong nSize = rNodeStack.size() - nElementCount;
+
         aRelationArray.resize(nSize);
         for (sal_uLong j=nSize;j > 0;j--)
         {
diff --git a/starmath/source/node.cxx b/starmath/source/node.cxx
index d8832a5..819bc5e 100644
--- a/starmath/source/node.cxx
+++ b/starmath/source/node.cxx
@@ -2521,7 +2521,8 @@ void SmMatrixNode::CreateTextFromNode(OUString &rText)
         for (sal_uInt16 j = 0;  j < nNumCols; j++)
         {
             SmNode *pNode = GetSubNode(i * nNumCols + j);
-            pNode->CreateTextFromNode(rText);
+            if (pNode)
+                pNode->CreateTextFromNode(rText);
             if (j != nNumCols-1)
                 rText += "# ";
         }


More information about the Libreoffice-commits mailing list