[Libreoffice-commits] core.git: Branch 'aoo/trunk' - sd/source

[Armin Le Grand]

 sd/source/ui/animations/CustomAnimationPane.cxx |   17 ++++++++++-------
 1 file changed, 10 insertions(+), 7 deletions(-)

New commits:
commit 966c38038352a347f25d690f18a2059d6f78b9fa
Author: Armin Le Grand <alg at apache.org>
Date:   Fri Jun 20 11:51:09 2014 +0000

    i121410 Secured usage of pTextGroup

diff --git a/sd/source/ui/animations/CustomAnimationPane.cxx b/sd/source/ui/animations/CustomAnimationPane.cxx
index 6622cda..1986212 100644
--- a/sd/source/ui/animations/CustomAnimationPane.cxx
+++ b/sd/source/ui/animations/CustomAnimationPane.cxx
@@ -1786,7 +1786,7 @@ void CustomAnimationPane::changeSelection( STLPropertySet* pResultSet, STLProper
 
             if( bHasAnimateForm )
             {
-                if( pTextGroup->getAnimateForm() != bAnimateForm )
+                if( pTextGroup.get() && pTextGroup->getAnimateForm() != bAnimateForm )
                 {
                     if( (pTextGroup->getTextGrouping() >= 0) && (nTextGrouping == -1 ) )
                     {
@@ -1804,22 +1804,25 @@ void CustomAnimationPane::changeSelection( STLPropertySet* pResultSet, STLProper
 
             if( bHasTextGrouping )
             {
-                if( (pTextGroup->getTextGrouping() != nTextGrouping) )
+                if( pTextGroup.get() && pTextGroup->getTextGrouping() != nTextGrouping )
                 {
                     pEffectSequence->setTextGrouping( pTextGroup, nTextGrouping );
                     bChanged = true;
                 }
             }
 
-            if (!bDoSetAnimateFormFirst&&bNeedDoSetAnimateForm)
+            if (!bDoSetAnimateFormFirst && bNeedDoSetAnimateForm)
             {
-                pEffectSequence->setAnimateForm( pTextGroup, bAnimateForm );
-                bChanged = true;
+                if( pTextGroup.get() )
+                {
+                    pEffectSequence->setAnimateForm( pTextGroup, bAnimateForm );
+                    bChanged = true;
+                }
             }
 
             if( bHasTextGroupingAuto )
             {
-                if( pTextGroup->getTextGroupingAuto() != fTextGroupingAuto )
+                if( pTextGroup.get() && pTextGroup->getTextGroupingAuto() != fTextGroupingAuto )
                 {
                     pEffectSequence->setTextGroupingAuto( pTextGroup, fTextGroupingAuto );
                     bChanged = true;
@@ -1828,7 +1831,7 @@ void CustomAnimationPane::changeSelection( STLPropertySet* pResultSet, STLProper
 
             if( bHasTextReverse )
             {
-                if( pTextGroup->getTextReverse() != bTextReverse )
+                if( pTextGroup.get() && pTextGroup->getTextReverse() != bTextReverse )
                 {
                     pEffectSequence->setTextReverse( pTextGroup, bTextReverse );
                     bChanged = true;