[Libreoffice-commits] core.git: cui/source
Norbert Thiebaud
nthiebaud at gmail.com
Thu Jun 27 05:41:53 PDT 2013
cui/source/tabpages/backgrnd.cxx | 12 ++++++++----
1 file changed, 8 insertions(+), 4 deletions(-)
New commits:
commit 3f8445b20e416ae913d68a31d522b948fe4b1684
Author: Norbert Thiebaud <nthiebaud at gmail.com>
Date: Wed Jun 26 18:13:09 2013 -0500
coverity#1038294 Explicit null dereference
Change-Id: Ifab48223b505a3061e70e5bc8741a293100a24bc
Reviewed-on: https://gerrit.libreoffice.org/4555
Reviewed-by: Michael Stahl <mstahl at redhat.com>
Tested-by: Michael Stahl <mstahl at redhat.com>
diff --git a/cui/source/tabpages/backgrnd.cxx b/cui/source/tabpages/backgrnd.cxx
index 16f8d95..ea37ea5 100644
--- a/cui/source/tabpages/backgrnd.cxx
+++ b/cui/source/tabpages/backgrnd.cxx
@@ -1728,12 +1728,16 @@ IMPL_LINK( SvxBackgroundTabPage, ParaDestinationHdl_Impl, ListBox*, pBox )
SvxBrushItem* pActItem = NULL;
switch(pParaBck_Impl->nActPos)
{
- case PARA_DEST_PARA:
- pActItem = pParaBck_Impl->pParaBrush;
+ case PARA_DEST_PARA:
+ pActItem = pParaBck_Impl->pParaBrush;
break;
- case PARA_DEST_CHAR:
- pActItem = pParaBck_Impl->pCharBrush;
+ case PARA_DEST_CHAR:
+ pActItem = pParaBck_Impl->pCharBrush;
break;
+ default:
+ /* we assert here because the rest of the code expect pActItem to be non NULL */
+ assert(false);
+ return 0;
}
pParaBck_Impl->nActPos = nSelPos;
if(XFILL_SOLID == lcl_getFillStyle(m_pLbSelect)) // brush selected
More information about the Libreoffice-commits
mailing list