[Libreoffice-commits] core.git: sd/source

[Caolán McNamara (via logerrit)]

 sd/source/ui/animations/CustomAnimationPane.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 7c5d207c6adaafa8c4f6fe90e3389c7fdaadc800
Author:     Caolán McNamara <caolanm at redhat.com>
AuthorDate: Sun Apr 5 20:33:31 2020 +0100
Commit:     Caolán McNamara <caolanm at redhat.com>
CommitDate: Sun Apr 5 22:29:30 2020 +0200

    attempt to deref end() iterator
    
    seen when dragging last animation in animation pane up one position
    
    Change-Id: I2c5163a58bb67fc7982c41546559207977e8b7c5
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91729
    Tested-by: Caolán McNamara <caolanm at redhat.com>
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/sd/source/ui/animations/CustomAnimationPane.cxx b/sd/source/ui/animations/CustomAnimationPane.cxx
index ea64b247f8fb..5b4ff943ec2e 100644
--- a/sd/source/ui/animations/CustomAnimationPane.cxx
+++ b/sd/source/ui/animations/CustomAnimationPane.cxx
@@ -2468,7 +2468,7 @@ void CustomAnimationPane::onDragNDropComplete(std::vector< CustomAnimationEffect
             mpMainSequence->moveToBeforeEffect( pEffect, pEffectInsertBefore );
 
             // Done moving effect and its hidden sub-effects when *next* effect is visible.
-            if ( mpCustomAnimationList->isVisible( *aIter ) )
+            if (aIter != aEnd && mpCustomAnimationList->isVisible(*aIter))
                 break;
         }