Land on Master: Fix two new / free mismatches
authorTom Sepez <tsepez@chromium.org>
Tue, 14 Apr 2015 23:00:03 +0000 (16:00 -0700)
committerTom Sepez <tsepez@chromium.org>
Tue, 14 Apr 2015 23:00:03 +0000 (16:00 -0700)
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

core/src/fpdfapi/fpdf_font/fpdf_font.cpp
core/src/fpdfdoc/doc_metadata.cpp

index 91d4010..9ee9621 100644 (file)
@@ -128,14 +128,12 @@ FX_BOOL CPDF_Font::Initialize()
 }
 CPDF_Font::~CPDF_Font()
 {
-    if (m_pCharMap) {
-        FX_Free(m_pCharMap);
-        m_pCharMap = NULL;
-    }
-    if (m_pToUnicodeMap) {
-        delete m_pToUnicodeMap;
-        m_pToUnicodeMap = NULL;
-    }
+    delete m_pCharMap;
+    m_pCharMap = NULL;
+
+    delete m_pToUnicodeMap;
+    m_pToUnicodeMap = NULL;
+
     if (m_pFontFile) {
         m_pDocument->GetPageData()->ReleaseFontFileStreamAcc((CPDF_Stream*)m_pFontFile->GetStream());
     }
index bf55527..232f6ee 100644 (file)
@@ -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);
 }