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

Julien Nabet serval2412 at yahoo.fr
Fri Aug 11 17:21:20 UTC 2017


 connectivity/source/drivers/macab/MacabRecords.cxx |    2 ++
 1 file changed, 2 insertions(+)

New commits:
commit 84fc4dec5f419cbe30fa6ac4e228f083837f7887
Author: Julien Nabet <serval2412 at yahoo.fr>
Date:   Fri Aug 11 19:09:24 2017 +0200

    Related tdf#111634: Multiple memory leaks in libmacabdrv1 (part2)
    
    Delete "sub arrays" of headerNames which is macabfield **
    before calling delete headerNames []
    
    Change-Id: I11b6d5483ed91eb2805b68ace1b06eafd23b707d
    Reviewed-on: https://gerrit.libreoffice.org/41054
    Reviewed-by: Julien Nabet <serval2412 at yahoo.fr>
    Tested-by: Julien Nabet <serval2412 at yahoo.fr>

diff --git a/connectivity/source/drivers/macab/MacabRecords.cxx b/connectivity/source/drivers/macab/MacabRecords.cxx
index d5e8d2c6f67f..ccc4dc3e8f6e 100644
--- a/connectivity/source/drivers/macab/MacabRecords.cxx
+++ b/connectivity/source/drivers/macab/MacabRecords.cxx
@@ -839,6 +839,8 @@ MacabHeader *MacabRecords::createHeaderForProperty(const ABPropertyType _propert
     {
         manageDuplicateHeaders(headerNames, length);
         MacabHeader *headerResult = new MacabHeader(length, headerNames);
+        for(sal_Int32 i = 0; i < length; ++i)
+            delete headerNames[i];
         delete [] headerNames;
         return headerResult;
     }


More information about the Libreoffice-commits mailing list