summaryrefslogtreecommitdiff
path: root/fpdfsdk/fpdfxfa/cpdfxfa_context.cpp
diff options
context:
space:
mode:
authorTom Sepez <tsepez@chromium.org>2017-05-16 15:57:26 -0700
committerChromium commit bot <commit-bot@chromium.org>2017-05-17 20:06:13 +0000
commita3cf977f974c67bd2cef8dee5b810784bfc21336 (patch)
tree7877cf8df0f5b6a214e870448e8d2df27a280e94 /fpdfsdk/fpdfxfa/cpdfxfa_context.cpp
parentb48912f4f3c428c8b71bbb714faa03cb843d40b0 (diff)
downloadpdfium-a3cf977f974c67bd2cef8dee5b810784bfc21336.tar.xz
CFX_UnownedPtr: check during assignment time as well.
In particular, doing m_pPtr = nullptr; in your dtor to evade this check will not longer work. Fix slight mis-ordering observeds in CFX_Font and CPDFXFA_Context. Change-Id: I3e6137159430333b091364021283a54a13d916b5 Reviewed-on: https://pdfium-review.googlesource.com/5570 Reviewed-by: Lei Zhang <thestig@chromium.org> Commit-Queue: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'fpdfsdk/fpdfxfa/cpdfxfa_context.cpp')
-rw-r--r--fpdfsdk/fpdfxfa/cpdfxfa_context.cpp3
1 files changed, 2 insertions, 1 deletions
diff --git a/fpdfsdk/fpdfxfa/cpdfxfa_context.cpp b/fpdfsdk/fpdfxfa/cpdfxfa_context.cpp
index 1b69ff45b9..1345bc8a08 100644
--- a/fpdfsdk/fpdfxfa/cpdfxfa_context.cpp
+++ b/fpdfsdk/fpdfxfa/cpdfxfa_context.cpp
@@ -67,9 +67,10 @@ CPDFXFA_Context::~CPDFXFA_Context() {
void CPDFXFA_Context::CloseXFADoc() {
if (!m_pXFADoc)
return;
+
+ m_pXFADocView = nullptr;
m_pXFADoc->CloseDoc();
m_pXFADoc.reset();
- m_pXFADocView = nullptr;
}
void CPDFXFA_Context::SetFormFillEnv(