sw/source/core/frmedt/feshview.cxx |    8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

New commits:
commit 8bf1abfdc6955004e034343a475ed0f9b9920b8d
Author:     Caolán McNamara <[email protected]>
AuthorDate: Wed Sep 11 09:44:29 2024 +0100
Commit:     Caolán McNamara <[email protected]>
CommitDate: Wed Sep 11 17:17:04 2024 +0200

    cid#1607403 silence Overflowed constant
    
    Change-Id: I5edaf48b4d6fb42d73bc34c2a028f5372ecbdc45
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/173190
    Tested-by: Caolán McNamara <[email protected]>
    Reviewed-by: Caolán McNamara <[email protected]>

diff --git a/sw/source/core/frmedt/feshview.cxx 
b/sw/source/core/frmedt/feshview.cxx
index 3220d3f44c2f..1dc47d379e81 100644
--- a/sw/source/core/frmedt/feshview.cxx
+++ b/sw/source/core/frmedt/feshview.cxx
@@ -2266,7 +2266,9 @@ bool SwFEShell::EndMark()
             SwFlyFrame* pOldSelFly = ::GetFlyFromMarked( &rMrkList, this );
 
             if ( rMrkList.GetMarkCount() > 1 )
-                for ( size_t i = 0; i < rMrkList.GetMarkCount(); ++i )
+            {
+                size_t i = 0;
+                while (i < rMrkList.GetMarkCount())
                 {
                     SdrObject *pObj = rMrkList.GetMark( i )->GetMarkedSdrObj();
                     if( dynamic_cast<const SwVirtFlyDrawObj*>( pObj) !=  
nullptr )
@@ -2276,9 +2278,11 @@ bool SwFEShell::EndMark()
                             bShowHdl = true;
                         }
                         rMrkList.DeleteMark( i );
-                        --i;    // no exceptions
+                        continue;
                     }
+                    ++i;
                 }
+            }
 
             if( bShowHdl )
             {

Reply via email to