From f105b19434fdeecac2dd2f78fed5b45ed2637ab3 Mon Sep 17 00:00:00 2001 From: Bo Xu Date: Thu, 28 Aug 2014 11:49:17 -0700 Subject: No need to release resources in destructor of CPDF_ShadingObject CPDF_DocPageData::~CPDF_DocPageData() will force to release all resources, so no need to do it here, which can result in heap-use-after-free trouble. BUG=408164 R=jun_fang@foxitsoftware.com, tsepez@chromium.org Review URL: https://codereview.chromium.org/513063003 --- core/src/fpdfapi/fpdf_page/fpdf_page.cpp | 4 ---- 1 file changed, 4 deletions(-) diff --git a/core/src/fpdfapi/fpdf_page/fpdf_page.cpp b/core/src/fpdfapi/fpdf_page/fpdf_page.cpp index 6d3cf29785..590a01aa6d 100644 --- a/core/src/fpdfapi/fpdf_page/fpdf_page.cpp +++ b/core/src/fpdfapi/fpdf_page/fpdf_page.cpp @@ -606,10 +606,6 @@ CPDF_ShadingObject::CPDF_ShadingObject() } CPDF_ShadingObject::~CPDF_ShadingObject() { - CPDF_ShadingPattern* pShading = m_pShading; - if (pShading && pShading->m_pDocument) { - pShading->m_pDocument->GetPageData()->ReleasePattern(pShading->m_pShadingObj); - } } void CPDF_ShadingObject::CopyData(const CPDF_PageObject* pSrc) { -- cgit v1.2.3