From d360477d8740f29e2c2bc5f7bbd667df7cd26ee9 Mon Sep 17 00:00:00 2001
From: Michael Stahl <mstahl@redhat.com>
Date: Tue, 31 Mar 2015 15:35:36 +0200
Subject: [PATCH] sw: fix obscure crash in SwXTextDocument::getSomething()

xNumFmtAgg may be null if you directly call global ServiceManager's
createInstanceWithContext("com.sun.star.text.TextDocument")

Change-Id: Id619a3f5c9e3f8281f9ef72db132c64287e027c4
---
 sw/source/uibase/uno/unotxdoc.cxx | 19 ++++++++-----------
 1 file changed, 8 insertions(+), 11 deletions(-)

diff --git a/sw/source/uibase/uno/unotxdoc.cxx b/sw/source/uibase/uno/unotxdoc.cxx
index 02c603cdddbf..f580db2a10cb 100644
--- a/sw/source/uibase/uno/unotxdoc.cxx
+++ b/sw/source/uibase/uno/unotxdoc.cxx
@@ -266,19 +266,16 @@ sal_Int64 SAL_CALL SwXTextDocument::getSomething( const Sequence< sal_Int8 >& rI
     }
 
     sal_Int64 nRet = SfxBaseModel::getSomething( rId );
-    if ( nRet )
+    if (nRet)
         return nRet;
-    else
-    {
-        GetNumberFormatter();
-        Any aNumTunnel = xNumFmtAgg->queryAggregation(cppu::UnoType<XUnoTunnel>::get());
-        Reference<XUnoTunnel> xNumTunnel;
-        aNumTunnel >>= xNumTunnel;
-        if(xNumTunnel.is())
-            return xNumTunnel->getSomething(rId);
-    }
 
-    return SfxBaseModel::getSomething( rId );
+    GetNumberFormatter();
+    if (!xNumFmtAgg.is()) // may happen if not valid or no SwDoc
+        return 0;
+    Any aNumTunnel = xNumFmtAgg->queryAggregation(cppu::UnoType<XUnoTunnel>::get());
+    Reference<XUnoTunnel> xNumTunnel;
+    aNumTunnel >>= xNumTunnel;
+    return (xNumTunnel.is()) ? xNumTunnel->getSomething(rId) : 0;
 }
 
 Any SAL_CALL SwXTextDocument::queryInterface( const uno::Type& rType ) throw(RuntimeException, std::exception)