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

[Julien Nabet (via logerrit)]

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

New commits:
commit 2f6ead197bbff5142199d00a30ab900ebb997754
Author:     Julien Nabet <serval2412 at yahoo.fr>
AuthorDate: Thu Jul 4 18:16:09 2019 +0200
Commit:     Julien Nabet <serval2412 at yahoo.fr>
CommitDate: Thu Jul 4 23:17:06 2019 +0200

    cid#1448425: Resource leak (sc/htmlprovider)
    
    Change-Id: I3c23280da75e51be37c4b66f14f805ca49ebf3cf
    Reviewed-on: https://gerrit.libreoffice.org/75095
    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 516a0b308376..bbe6dd3c514b 100644
--- a/sc/source/ui/dataprovider/htmldataprovider.cxx
+++ b/sc/source/ui/dataprovider/htmldataprovider.cxx
@@ -195,7 +195,12 @@ void HTMLFetchThread::execute()
     xmlNodeSetPtr pXmlNodes = pXmlXpathObj->nodesetval;
 
     if (pXmlNodes->nodeNr == 0)
+    {
+        xmlXPathFreeNodeSet(pXmlNodes);
+        xmlXPathFreeNodeSetList(pXmlXpathObj);
+        xmlXPathFreeContext(pXmlXpathCtx);
         return;
+    }
 
     xmlNodePtr pNode = pXmlNodes->nodeTab[0];
     handleTable(pNode);