diff options
| author | Julien Nabet <serval2412@yahoo.fr> | 2017-08-11 19:09:24 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-08-14 08:34:06 +0200 |
| commit | 39a58df9e9f990faf231c2b1c2dc7fcdc1c4a6cf (patch) | |
| tree | 37ac0fffabe7bf12dcdf00758133ff3b00dfbbea | |
| parent | c68a7c94f02989ac7122be0a319251dbaf344952 (diff) | |
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@yahoo.fr>
Tested-by: Julien Nabet <serval2412@yahoo.fr>
(cherry picked from commit 84fc4dec5f419cbe30fa6ac4e228f083837f7887)
Reviewed-on: https://gerrit.libreoffice.org/41068
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
| -rw-r--r-- | connectivity/source/drivers/macab/MacabRecords.cxx | 2 |
1 files changed, 2 insertions, 0 deletions
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; } |