[Libreoffice-commits] core.git: sd/source
Caolán McNamara (via logerrit)
logerrit at kemper.freedesktop.org
Fri Jul 26 15:37:28 UTC 2019
sd/source/ui/func/fupage.cxx | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
New commits:
commit f9190ae607a5766a087b8ed5d7ac29ec8a3272c9
Author: Caolán McNamara <caolanm at redhat.com>
AuthorDate: Fri Jul 26 09:22:53 2019 +0100
Commit: Caolán McNamara <caolanm at redhat.com>
CommitDate: Fri Jul 26 17:36:27 2019 +0200
cid#1448362 silence Dereference null return value
Change-Id: I6b24a10bb2cd67d9750702a43d15162fb5347d76
Reviewed-on: https://gerrit.libreoffice.org/76366
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm at redhat.com>
Tested-by: Caolán McNamara <caolanm at redhat.com>
diff --git a/sd/source/ui/func/fupage.cxx b/sd/source/ui/func/fupage.cxx
index ac3512ad5165..ae543ff9464f 100644
--- a/sd/source/ui/func/fupage.cxx
+++ b/sd/source/ui/func/fupage.cxx
@@ -370,7 +370,9 @@ const SfxItemSet* FuPage::ExecuteDialog(weld::Window* pParent, SfxRequest& rReq)
}
// if the background for this page was set to invisible, the background-object has to be deleted, too.
- if( ( pTempSet->GetItem<XFillStyleItem>( XATTR_FILLSTYLE )->GetValue() == drawing::FillStyle_NONE ) ||
+ const XFillStyleItem* pTempFillStyleItem = pTempSet->GetItem<XFillStyleItem>(XATTR_FILLSTYLE);
+ assert(pTempFillStyleItem);
+ if( ( pTempFillStyleItem->GetValue() == drawing::FillStyle_NONE ) ||
( ( pTempSet->GetItemState( XATTR_FILLSTYLE ) == SfxItemState::DEFAULT ) &&
( aMergedAttr.GetItem<XFillStyleItem>( XATTR_FILLSTYLE )->GetValue() == drawing::FillStyle_NONE ) ) )
mbPageBckgrdDeleted = true;
More information about the Libreoffice-commits
mailing list