diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2014-10-23 16:27:59 +0100 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2014-10-24 09:24:52 +0100 |
| commit | 07ff3d965f10a68bcae93ec216c269fbb7009a6b (patch) | |
| tree | df4850288cb728a2e71c38cee5c00f45d3f3b24b | |
| parent | 1304a6ae7e60a46a88ded646bb2af8b8f8b926df (diff) | |
coverity#703957 Unchecked return value
Change-Id: I3a5c0239f4615ff0ba832ac4c733c26e25c259e6
| -rw-r--r-- | filter/source/msfilter/escherex.cxx | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/filter/source/msfilter/escherex.cxx b/filter/source/msfilter/escherex.cxx index afcc626d3ba2..e370e5bcc08a 100644 --- a/filter/source/msfilter/escherex.cxx +++ b/filter/source/msfilter/escherex.cxx @@ -1249,14 +1249,12 @@ bool EscherPropertyContainer::CreateShapeProperties( const ::com::sun::star::uno bool bVal = false; ::com::sun::star::uno::Any aAny; sal_uInt32 nShapeAttr = 0; - (void)EscherPropertyValueHelper::GetPropertyValue( aAny, aXPropSet, OUString( "Visible" ), true ); - if ( aAny >>= bVal ) + if (EscherPropertyValueHelper::GetPropertyValue(aAny, aXPropSet, OUString("Visible"), true) && (aAny >>= bVal)) { if ( !bVal ) nShapeAttr |= 0x20002; // set fHidden = true } - (void)EscherPropertyValueHelper::GetPropertyValue( aAny, aXPropSet, OUString( "Printable" ), true ); - if ( aAny >>= bVal ) + if (EscherPropertyValueHelper::GetPropertyValue(aAny, aXPropSet, OUString( "Printable"), true) && (aAny >>= bVal)) { if ( !bVal ) nShapeAttr |= 0x10000; // set fPrint = false; |