summaryrefslogtreecommitdiff
path: root/sw
diff options
context:
space:
mode:
authorCaolán McNamara <caolanm@redhat.com>2015-10-14 10:01:06 +0100
committerCaolán McNamara <caolanm@redhat.com>2015-10-14 10:59:46 +0100
commit6dd3c3558291f8b23e339efd967a1ef76021b1cf (patch)
tree4461e9aab50cfc4f3571c1f237a77b5df4fcd123 /sw
parentd7b75ef4ff20318c16ede48b7daa7b8c3240496d (diff)
coverity#735965 Unchecked dynamic_cast
Change-Id: Iaeccbd37ae67a6893690d2cff5897678b81b23a0
Diffstat (limited to 'sw')
-rw-r--r--sw/source/filter/xml/xmlexpit.cxx13
1 files changed, 6 insertions, 7 deletions
diff --git a/sw/source/filter/xml/xmlexpit.cxx b/sw/source/filter/xml/xmlexpit.cxx
index bc58820ca82e..3753d2e84029 100644
--- a/sw/source/filter/xml/xmlexpit.cxx
+++ b/sw/source/filter/xml/xmlexpit.cxx
@@ -1071,33 +1071,32 @@ bool SvXMLExportItemMapper::QueryXMLValue(
case RES_FRM_SIZE:
{
- const SwFormatFrmSize* pFrmSize = dynamic_cast<const SwFormatFrmSize*>( &rItem );
- OSL_ENSURE( pFrmSize != NULL, "Wrong Which-ID" );
+ const SwFormatFrmSize& rFrmSize = dynamic_cast<const SwFormatFrmSize&>(rItem);
bool bOutHeight = false;
switch( nMemberId )
{
case MID_FRMSIZE_REL_WIDTH:
- if( pFrmSize->GetWidthPercent() )
+ if (rFrmSize.GetWidthPercent())
{
::sax::Converter::convertPercent(
- aOut, pFrmSize->GetWidthPercent() );
+ aOut, rFrmSize.GetWidthPercent() );
bOk = true;
}
break;
case MID_FRMSIZE_MIN_HEIGHT:
- if( ATT_MIN_SIZE == pFrmSize->GetHeightSizeType() )
+ if( ATT_MIN_SIZE == rFrmSize.GetHeightSizeType() )
bOutHeight = true;
break;
case MID_FRMSIZE_FIX_HEIGHT:
- if( ATT_FIX_SIZE == pFrmSize->GetHeightSizeType() )
+ if( ATT_FIX_SIZE == rFrmSize.GetHeightSizeType() )
bOutHeight = true;
break;
}
if( bOutHeight )
{
- rUnitConverter.convertMeasureToXML(aOut, pFrmSize->GetHeight());
+ rUnitConverter.convertMeasureToXML(aOut, rFrmSize.GetHeight());
bOk = true;
}
}