Fix a crashier in CPDFSDK_Document::GetPageView() BUG=pdfium:369 R=tsepez@chromium.org Review URL: https://codereview.chromium.org/1667643002 .
diff --git a/fpdfsdk/src/fpdfxfa/fpdfxfa_doc.cpp b/fpdfsdk/src/fpdfxfa/fpdfxfa_doc.cpp index b8549fc..b937af4 100644 --- a/fpdfsdk/src/fpdfxfa/fpdfxfa_doc.cpp +++ b/fpdfsdk/src/fpdfxfa/fpdfxfa_doc.cpp
@@ -45,18 +45,6 @@ } CPDFXFA_Document::~CPDFXFA_Document() { - if (m_pJSContext && m_pSDKDoc && m_pSDKDoc->GetEnv()) - m_pSDKDoc->GetEnv()->GetJSRuntime()->ReleaseContext(m_pJSContext); - - delete m_pSDKDoc; - - if (m_pPDFDoc) { - CPDF_Parser* pParser = m_pPDFDoc->GetParser(); - if (pParser) - delete pParser; - else - delete m_pPDFDoc; - } if (m_pXFADoc) { IXFA_App* pApp = m_pApp->GetXFAApp(); if (pApp) { @@ -67,6 +55,16 @@ } delete m_pXFADoc; } + if (m_pJSContext && m_pSDKDoc && m_pSDKDoc->GetEnv()) + m_pSDKDoc->GetEnv()->GetJSRuntime()->ReleaseContext(m_pJSContext); + delete m_pSDKDoc; + if (m_pPDFDoc) { + CPDF_Parser* pParser = m_pPDFDoc->GetParser(); + if (pParser) + delete pParser; + else + delete m_pPDFDoc; + } } FX_BOOL CPDFXFA_Document::LoadXFADoc() { @@ -490,7 +488,6 @@ if (!pPage) return; pEnv->FFI_PageEvent(pPage->GetPageIndex(), dwFlags); - m_pSDKDoc->GetPageView(pPage)->ClearFXAnnots(); pPage->Release(); }