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

[Julien Nabet (via logerrit)]

 sc/source/ui/dataprovider/htmldataprovider.cxx |   13 +++++++++++++
 1 file changed, 13 insertions(+)

New commits:
commit ec57f6eca37192ada29cda1d37ec192472c0d9ef
Author:     Julien Nabet <serval2412 at yahoo.fr>
AuthorDate: Tue Sep 10 21:16:43 2019 +0200
Commit:     Julien Nabet <serval2412 at yahoo.fr>
CommitDate: Wed Sep 11 09:59:54 2019 +0200

    tdf#127461: fix data provider crashes
    
    See bts provided in the bugtracker
    
    Change-Id: I1cc38add93446f8372c80ca45962b04bfa627923
    Reviewed-on: https://gerrit.libreoffice.org/78803
    Tested-by: Jenkins
    Reviewed-by: Julien Nabet <serval2412 at yahoo.fr>

diff --git a/sc/source/ui/dataprovider/htmldataprovider.cxx b/sc/source/ui/dataprovider/htmldataprovider.cxx
index bbe6dd3c514b..b8b7b3746d2c 100644
--- a/sc/source/ui/dataprovider/htmldataprovider.cxx
+++ b/sc/source/ui/dataprovider/htmldataprovider.cxx
@@ -192,8 +192,21 @@ void HTMLFetchThread::execute()
     OString aID = OUStringToOString(maID, RTL_TEXTENCODING_UTF8);
     xmlXPathContextPtr pXmlXpathCtx = xmlXPathNewContext(pHtmlPtr);
     xmlXPathObjectPtr pXmlXpathObj = xmlXPathEvalExpression(BAD_CAST(aID.getStr()), pXmlXpathCtx);
+
+    if (!pXmlXpathObj)
+    {
+        xmlXPathFreeContext(pXmlXpathCtx);
+        return;
+    }
     xmlNodeSetPtr pXmlNodes = pXmlXpathObj->nodesetval;
 
+    if (!pXmlNodes)
+    {
+        xmlXPathFreeNodeSetList(pXmlXpathObj);
+        xmlXPathFreeContext(pXmlXpathCtx);
+        return;
+    }
+
     if (pXmlNodes->nodeNr == 0)
     {
         xmlXPathFreeNodeSet(pXmlNodes);