summaryrefslogtreecommitdiff
path: root/core/src/fpdfdoc/doc_metadata.cpp
diff options
context:
space:
mode:
authorTom Sepez <tsepez@chromium.org>2015-04-14 16:00:03 -0700
committerTom Sepez <tsepez@chromium.org>2015-04-14 16:00:03 -0700
commitb48704f27fa425d0c4f5b0040cc11e4087c39016 (patch)
tree64708fe85e56c1a7c8e99ddb48c9c127c001f728 /core/src/fpdfdoc/doc_metadata.cpp
parent4926900ab54a493d236291b5a24dfa4476792182 (diff)
downloadpdfium-b48704f27fa425d0c4f5b0040cc11e4087c39016.tar.xz
Land on Master: Fix two new / free mismatches
Note: Landed on XFA first by mistake. Original review: https://codereview.chromium.org/1085963002/ TBR=brucedawson@chromium.org Review URL: https://codereview.chromium.org/1090463002
Diffstat (limited to 'core/src/fpdfdoc/doc_metadata.cpp')
-rw-r--r--core/src/fpdfdoc/doc_metadata.cpp8
1 files changed, 3 insertions, 5 deletions
diff --git a/core/src/fpdfdoc/doc_metadata.cpp b/core/src/fpdfdoc/doc_metadata.cpp
index bf55527d62..232f6eea21 100644
--- a/core/src/fpdfdoc/doc_metadata.cpp
+++ b/core/src/fpdfdoc/doc_metadata.cpp
@@ -28,7 +28,7 @@ CPDF_Metadata::CPDF_Metadata()
{
m_pData = FX_Alloc(PDFDOC_METADATA, 1);
CFX_CMapByteStringToPtr *&pStringMap = ((PDFDOC_LPMETADATA)m_pData)->m_pStringMap;
- pStringMap = FX_NEW(CFX_CMapByteStringToPtr);
+ pStringMap = FX_NEW CFX_CMapByteStringToPtr;
if (pStringMap != NULL) {
CFX_ByteString bstr;
for (int i = 0; i < 18; i += 2) {
@@ -41,13 +41,11 @@ CPDF_Metadata::~CPDF_Metadata()
{
FXSYS_assert(m_pData != NULL);
CXML_Element *&p = ((PDFDOC_LPMETADATA)m_pData)->m_pXmlElmnt;
- if (p) {
- delete p;
- }
+ delete p;
CFX_CMapByteStringToPtr *pStringMap = ((PDFDOC_LPMETADATA)m_pData)->m_pStringMap;
if (pStringMap) {
pStringMap->RemoveAll();
- FX_Free(pStringMap);
+ delete pStringMap;
}
FX_Free(m_pData);
}