diff options
| author | Tom Sepez <tsepez@chromium.org> | 2015-04-14 16:00:03 -0700 |
|---|---|---|
| committer | Tom Sepez <tsepez@chromium.org> | 2015-04-14 16:00:03 -0700 |
| commit | b48704f27fa425d0c4f5b0040cc11e4087c39016 (patch) | |
| tree | 64708fe85e56c1a7c8e99ddb48c9c127c001f728 /core/src/fpdfdoc/doc_metadata.cpp | |
| parent | 4926900ab54a493d236291b5a24dfa4476792182 (diff) | |
| download | pdfium-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.cpp | 8 |
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); } |