free, rather than delete, malloc'd object.

core/src/fxcodec/jbig2/JBig2_Context.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 "JBig2_Context.h"
 void OutputBitmap(CJBig2_Image* pImage)
 {
     if(!pImage) {
@@ skipping 41 matching lines @@
 {
     if(m_pArithDecoder) {
         delete m_pArithDecoder;
     }
     m_pArithDecoder = NULL;
     if(m_pGRD) {
         delete m_pGRD;
     }
     m_pGRD = NULL;
     if(m_gbContext) {
-        delete m_gbContext;
+        m_pModule->JBig2_Free(m_gbContext);

[jam, 2014/06/05 23:50:23]

seems that this is a problem with other member variables in this class?

[jun_fang, 2014/06/09 01:26:24]

A different way, JBIG2_ALLOC, is used for memory allocation of the other member
variables in this de-construction. It is a wrapper of 'new'. It's fine to use
'delete' to release memory for them. The variable m_gbcontext in this issue
points to the memory allocated by JBig2_Malloc which is internally implemented
in PDF SDK. It needs to call JBig2_Free to release memory.

     }
     m_gbContext = NULL;
     if(m_pGlobalContext) {
         delete m_pGlobalContext;
     }
     m_pGlobalContext = NULL;
     if(m_pPageInfoList) {
         delete m_pPageInfoList;
     }
     m_pPageInfoList = NULL;
@@ skipping 1730 matching lines @@
                 SBSYMCODES[CURTEMP].code = CURCODE;
                 CURCODE = CURCODE + 1;
             }
             CURTEMP = CURTEMP + 1;
         }
         CURLEN = CURLEN + 1;
     }
     m_pModule->JBig2_Free(LENCOUNT);
     m_pModule->JBig2_Free(FIRSTCODE);
 }