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

Caolán McNamara caolanm at redhat.com
Sun Dec 31 14:20:30 UTC 2017


 sw/source/filter/html/swhtml.cxx |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

New commits:
commit d4d0b1a2b9f11fb7629559e08c345697ba129c04
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Sun Dec 31 14:19:34 2017 +0000

    ofz#4848 Null-dereference READ
    
    Change-Id: I632aca7e3f59b7edf48c8c35eff4abf1946af652

diff --git a/sw/source/filter/html/swhtml.cxx b/sw/source/filter/html/swhtml.cxx
index 7376ab23da47..d6ffd1928348 100644
--- a/sw/source/filter/html/swhtml.cxx
+++ b/sw/source/filter/html/swhtml.cxx
@@ -1462,10 +1462,10 @@ void SwHTMLParser::NextToken( HtmlTokenId nToken )
         if( !aToken.isEmpty() && ' '==aToken[0] && !IsReadPRE() )
         {
             sal_Int32 nPos = m_pPam->GetPoint()->nContent.GetIndex();
-            if( nPos )
+            const SwTextNode* pTextNode = nPos ? m_pPam->GetPoint()->nNode.GetNode().GetTextNode() : nullptr;
+            if (pTextNode)
             {
-                const OUString& rText =
-                    m_pPam->GetPoint()->nNode.GetNode().GetTextNode()->GetText();
+                const OUString& rText = pTextNode->GetText();
                 sal_Unicode cLast = rText[--nPos];
                 if( ' ' == cLast || '\x0a' == cLast)
                     aToken = aToken.copy(1);


More information about the Libreoffice-commits mailing list