Resolves: #i121410# Secured usage of pTextGroup

(cherry picked from commit 966c38038352a347f25d690f18a2059d6f78b9fa)

Change-Id: Ibbd68e8c102f2d80ea5b7610b9a97704a282e6ea
This commit is contained in:
Armin Le Grand 2014-06-20 11:51:09 +00:00 committed by Caolán McNamara
parent 465503f52d
commit 553f88e3d6

View File

@ -1500,7 +1500,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 ) )
{
@ -1518,22 +1518,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;
@ -1542,7 +1545,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;