diff options
author | Bo Xu <bo_xu@foxitsoftware.com> | 2014-07-21 10:02:07 -0700 |
---|---|---|
committer | Bo Xu <bo_xu@foxitsoftware.com> | 2014-07-21 10:02:07 -0700 |
commit | 22b31e0b3d14a50c4dd70ed22e0f24975dc1137c (patch) | |
tree | 70be27446d563f5508eb3a716ce52264d92223cf /core/src/fpdfdoc/doc_formfield.cpp | |
parent | baca77096e21acb07fa0b6e3a9ca7b926d18dcf1 (diff) | |
download | pdfium-22b31e0b3d14a50c4dd70ed22e0f24975dc1137c.tar.xz |
Adjust null object checkchromium/2103chromium/2102
BUG=None
R=thakis@chromium.org
Review URL: https://codereview.chromium.org/396173003
Diffstat (limited to 'core/src/fpdfdoc/doc_formfield.cpp')
-rw-r--r-- | core/src/fpdfdoc/doc_formfield.cpp | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/core/src/fpdfdoc/doc_formfield.cpp b/core/src/fpdfdoc/doc_formfield.cpp index e51acf94a1..eeba372390 100644 --- a/core/src/fpdfdoc/doc_formfield.cpp +++ b/core/src/fpdfdoc/doc_formfield.cpp @@ -1077,9 +1077,12 @@ FX_BOOL CPDF_FormField::ClearSelectedOptions(FX_BOOL bNotify) } void CPDF_FormField::LoadDA() { - CFX_ByteString DA = FPDF_GetFieldAttr(m_pDict, "DA") ? FPDF_GetFieldAttr(m_pDict, "DA")->GetString() : CFX_ByteString(); - if (DA.IsEmpty()) { - DA = m_pForm->m_pFormDict ? m_pForm->m_pFormDict->GetString("DA") : CFX_ByteString(); + CFX_ByteString DA; + if (CPDF_Object* pObj_t = FPDF_GetFieldAttr(m_pDict, "DA")) { + DA = pObj_t->GetString(); + } + if (DA.IsEmpty() && m_pForm->m_pFormDict) { + DA = m_pForm->m_pFormDict->GetString("DA"); } if (DA.IsEmpty()) { return; |