CFX_MapByteStringToPtr considered harmful.

core/src/fpdfapi/fpdf_font/fpdf_font_cid.cpp

 // Copyright 2014 PDFium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
 
 #include "../../../include/fpdfapi/fpdf_module.h"
 #include "../../../include/fpdfapi/fpdf_page.h"
 #include "../../../include/fpdfapi/fpdf_resource.h"
 #include "../../../include/fxge/fx_freetype.h"
 #include "../../../include/fxge/fx_ge.h"
 #include "../fpdf_cmaps/cmap_int.h"
 #include "font_int.h"
 
 extern short TT2PDF(int m, FXFT_Face face);
 extern FX_BOOL FT_UseTTCharmap(FXFT_Face face,
                                int platform_id,
                                int encoding_id);
 
 CPDF_CMapManager::CPDF_CMapManager() {
   m_bPrompted = FALSE;
   FXSYS_memset(m_CID2UnicodeMaps, 0, sizeof m_CID2UnicodeMaps);
 }
 CPDF_CMapManager::~CPDF_CMapManager() {
-  DropAll(FALSE);
+  auto it = m_CMaps.begin();
+  while (it != m_CMaps.end()) {

[Lei Zhang, 2015/08/15 00:35:44]

Range-Based For Loop?

[Tom Sepez, 2015/08/17 20:15:25]

Done; you're right its probably faster to just clear it at the end despite the
icky feeling of temporarily leaving dangling pointers in the map.

+    auto temp = it++;
+    delete temp->second;
+    m_CMaps.erase(temp);
+  }
+  for (int i = 0; i < FX_ArraySize(m_CID2UnicodeMaps); ++i) {
+    delete m_CID2UnicodeMaps[i];
+  }
 }
 CPDF_CMap* CPDF_CMapManager::GetPredefinedCMap(const CFX_ByteString& name,
                                                FX_BOOL bPromptCJK) {
-  CPDF_CMap* pCMap;
-  if (m_CMaps.Lookup(name, (void*&)pCMap)) {
-    return pCMap;
+  auto it = m_CMaps.find(name);
+  if (it != m_CMaps.end()) {
+    return it->second;
   }
-  pCMap = LoadPredefinedCMap(name, bPromptCJK);
-  if (name.IsEmpty()) {
-    return pCMap;
+  CPDF_CMap* pCMap = LoadPredefinedCMap(name, bPromptCJK);
+  if (!name.IsEmpty()) {
+    m_CMaps[name] = pCMap;
   }
-  m_CMaps.SetAt(name, pCMap);
   return pCMap;
 }
 CPDF_CMap* CPDF_CMapManager::LoadPredefinedCMap(const CFX_ByteString& name,
                                                 FX_BOOL bPromptCJK) {
   CPDF_CMap* pCMap = new CPDF_CMap;
   const FX_CHAR* pname = name;
   if (*pname == '/') {
     pname++;
   }
   pCMap->LoadPredefined(this, pname, bPromptCJK);
   return pCMap;
 }
 static const FX_CHAR* const g_CharsetNames[NUMBER_OF_CIDSETS] = {
     NULL, "GB1", "CNS1", "Japan1", "Korea1", "UCS"};
 static const int g_CharsetCPs[NUMBER_OF_CIDSETS] = {0,   936, 950,
                                                     932, 949, 1200};
 int _CharsetFromOrdering(const CFX_ByteString& Ordering) {
   for (int charset = 1; charset < NUMBER_OF_CIDSETS; charset++) {
     if (Ordering == CFX_ByteStringC(g_CharsetNames[charset]))
       return charset;
   }
   return CIDSET_UNKNOWN;
 }
 void CPDF_CMapManager::ReloadAll() {
-  DropAll(TRUE);
-}
-void CPDF_CMapManager::DropAll(FX_BOOL bReload) {
-  FX_POSITION pos = m_CMaps.GetStartPosition();
-  while (pos) {
-    CFX_ByteString name;
-    CPDF_CMap* pCMap;
-    m_CMaps.GetNextAssoc(pos, name, (void*&)pCMap);
-    if (pCMap == NULL) {
-      continue;
-    }
-    if (bReload) {
-      pCMap->LoadPredefined(this, name, FALSE);
-    } else {
-      delete pCMap;
+  for (auto it = m_CMaps.begin(); it != m_CMaps.end(); ++it) {
+    if (CPDF_CMap* pCMap = it->second) {

[Lei Zhang, 2015/08/15 00:35:44]

This is always true.

[Tom Sepez, 2015/08/17 20:15:25]

Done.

+      pCMap->LoadPredefined(this, it->first, FALSE);
     }
   }
-  for (int i = 0; i < sizeof m_CID2UnicodeMaps / sizeof(CPDF_CID2UnicodeMap*);
-       i++) {
-    CPDF_CID2UnicodeMap* pMap = m_CID2UnicodeMaps[i];
-    if (pMap == NULL) {
-      continue;
-    }
-    if (bReload) {
+  for (int i = 0; i < FX_ArraySize(m_CID2UnicodeMaps); ++i) {
+    if (CPDF_CID2UnicodeMap* pMap = m_CID2UnicodeMaps[i]) {
       pMap->Load(this, i, FALSE);
-    } else {
-      delete pMap;
     }
   }
 }
 CPDF_CID2UnicodeMap* CPDF_CMapManager::GetCID2UnicodeMap(int charset,
                                                          FX_BOOL bPromptCJK) {
   if (m_CID2UnicodeMaps[charset] == NULL) {
     m_CID2UnicodeMaps[charset] = LoadCID2UnicodeMap(charset, bPromptCJK);
   }
   return m_CID2UnicodeMaps[charset];
 }
@@ skipping 1557 matching lines @@
     if (middlecode > CID) {
       end = middle - 1;
     } else if (middlecode < CID) {
       begin = middle + 1;
     } else {
       return &Japan1_VertCIDs[middle].a;
     }
   }
   return NULL;
 }