diff options
author | Caolán McNamara <caolanm@redhat.com> | 2017-07-27 21:23:49 +0100 |
---|---|---|
committer | Michael Stahl <mstahl@redhat.com> | 2017-07-28 11:01:34 +0200 |
commit | 26222c3baca76eaed5f45dde4592a059e0e4a85d (patch) | |
tree | 4986d7d08ed42a2a38059fc3a75a812121cf0721 | |
parent | 3cb369463e4e9d0a00baee3adf10957d1ed7ed83 (diff) |
ofz#2759 null dereference
(cherry picked from commit 6f5841e60ed29ae2577e63623edacc9fe1467ba5)
Change-Id: I23671f0cea592c92a05b34b3cf284a47a73962b1
Reviewed-on: https://gerrit.libreoffice.org/40506
Reviewed-by: Michael Stahl <mstahl@redhat.com>
Tested-by: Jenkins <ci@libreoffice.org>
-rw-r--r-- | sw/source/filter/ww8/ww8par2.cxx | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/sw/source/filter/ww8/ww8par2.cxx b/sw/source/filter/ww8/ww8par2.cxx index bcc424d4fc83..2b4bccb33189 100644 --- a/sw/source/filter/ww8/ww8par2.cxx +++ b/sw/source/filter/ww8/ww8par2.cxx @@ -409,14 +409,16 @@ long SwWW8ImplReader::Read_Footnote(WW8PLCFManResult* pRes) if (eEDN == pRes->nSprmId) { aDesc.meType = MAN_EDN; - if (m_pPlcxMan->GetEdn()) - aDesc.mbAutoNum = 0 != *static_cast<short const *>(m_pPlcxMan->GetEdn()->GetData()); + WW8PLCFx_SubDoc* pEndNote = m_pPlcxMan->GetEdn(); + if (const void* pData = pEndNote ? pEndNote->GetData() : nullptr) + aDesc.mbAutoNum = 0 != *static_cast<short const*>(pData); } else { aDesc.meType = MAN_FTN; - if (m_pPlcxMan->GetFootnote()) - aDesc.mbAutoNum = 0 != *static_cast<short const *>(m_pPlcxMan->GetFootnote()->GetData()); + WW8PLCFx_SubDoc* pFootNote = m_pPlcxMan->GetFootnote(); + if (const void* pData = pFootNote ? pFootNote->GetData() : nullptr) + aDesc.mbAutoNum = 0 != *static_cast<short const*>(pData); } aDesc.mnStartCp = pRes->nCp2OrIdx; |