diff options
author | Caolán McNamara <caolanm@redhat.com> | 2014-02-27 10:26:54 +0000 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2014-02-27 12:58:26 +0000 |
commit | d9404f150d9389eead85498a12e1f20dcccb6d5e (patch) | |
tree | 259c9ed1d0bc3a95e38de9cc289257492b7376f2 /sd | |
parent | coverity#1187660 Dereference after null check (diff) | |
download | core-d9404f150d9389eead85498a12e1f20dcccb6d5e.tar.gz core-d9404f150d9389eead85498a12e1f20dcccb6d5e.zip |
coverity#1187661 Unchecked dynamic_cast
Change-Id: I66f8382c7dd7ab8d115204fe45d6a33ff77adb78
Diffstat (limited to 'sd')
-rw-r--r-- | sd/source/ui/unoidl/unoobj.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sd/source/ui/unoidl/unoobj.cxx b/sd/source/ui/unoidl/unoobj.cxx index 75c997f5f23d..bfd6b4a2cbea 100644 --- a/sd/source/ui/unoidl/unoobj.cxx +++ b/sd/source/ui/unoidl/unoobj.cxx @@ -544,9 +544,9 @@ void SAL_CALL SdXShape::setPropertyValue( const OUString& aPropertyName, const : if(bIsAnimation) { SdrObjGroup* pGroup = dynamic_cast< SdrObjGroup* >(pObj); - SdPage* pPage = dynamic_cast< SdPage* >(pGroup->GetPage()); + SdPage* pPage = pGroup ? dynamic_cast< SdPage* >(pGroup->GetPage()) : NULL; - if(pGroup && pPage) + if (pPage) { // #i42894# Animated Group object, migrate that effect EffectMigration::CreateAnimatedGroup(*pGroup, *pPage); |