diff options
author | Caolán McNamara <caolanm@redhat.com> | 2020-02-01 12:04:26 +0000 |
---|---|---|
committer | Andras Timar <andras.timar@collabora.com> | 2021-05-06 12:47:18 +0200 |
commit | 75311553cb0dfebeb8fc9b408946e5e12f5dea98 (patch) | |
tree | 23ac7ea3e9bcac3941afca952ac7dc4db0efab37 | |
parent | 13657cbe25c4de477d2b39b8662383c278f1191e (diff) |
ofz#20456 Null-dereference READ
this is the first honggfuzz (honggfuzz_asan_libreoffice) report I've seen to date
Change-Id: Iac733aa63c7c94d9454b9c8596340dc2286393c0
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/87803
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
(cherry picked from commit 0754e581b0d8569dd08cf26f88678754f249face)
-rw-r--r-- | lotuswordpro/source/filter/lwprowlayout.cxx | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/lotuswordpro/source/filter/lwprowlayout.cxx b/lotuswordpro/source/filter/lwprowlayout.cxx index 287f8de65434..10eed55d0499 100644 --- a/lotuswordpro/source/filter/lwprowlayout.cxx +++ b/lotuswordpro/source/filter/lwprowlayout.cxx @@ -282,14 +282,15 @@ void LwpRowLayout::RegisterCurRowStyle(XFRow* pXFRow,sal_uInt16 nRowMark) { pRowStyle = static_cast<XFRowStyle*>( pXFStyleManager->FindStyle(pTableLayout->GetDefaultRowStyleName())); - fHeight += pRowStyle->GetRowHeight(); } else { pRowStyle = static_cast<XFRowStyle*>( pXFStyleManager->FindStyle(iter->second->GetStyleName())); - fHeight+=pRowStyle->GetRowHeight(); } + if (!pRowStyle) + throw std::runtime_error("missing RowStyle"); + fHeight += pRowStyle->GetRowHeight(); } if (m_nDirection & 0x0030) |