core.git: sw/source

[Caolán McNamara (via logerrit)]

 sw/source/filter/ww8/wrtw8esh.cxx |    7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

New commits:
commit 7c76f391296ef129e89c724bff26cc1bd3aefa27
Author:     Caolán McNamara <caolan.mcnam...@collabora.com>
AuthorDate: Sun Oct 20 15:02:04 2024 +0100
Commit:     Caolán McNamara <caolan.mcnam...@collabora.com>
CommitDate: Tue Oct 22 11:55:20 2024 +0200

    cid#1607845 silence Overflowed constant
    
    Change-Id: I97af2c10eab0ecc3c00416e1e7263b779964def8
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175375
    Reviewed-by: Caolán McNamara <caolan.mcnam...@collabora.com>
    Tested-by: Jenkins

diff --git a/sw/source/filter/ww8/wrtw8esh.cxx 
b/sw/source/filter/ww8/wrtw8esh.cxx
index c1066d355569..69707aa46c24 100644
--- a/sw/source/filter/ww8/wrtw8esh.cxx
+++ b/sw/source/filter/ww8/wrtw8esh.cxx
@@ -2208,8 +2208,10 @@ SwEscherEx::SwEscherEx(SvStream* pStrm, WW8Export& 
rWW8Wrt)
         m_pTextBxs = mrWrt.m_pTextBxs.get();
     }
 
-    for( ; i--; pSdrObjs = mrWrt.m_pSdrObjs.get(), m_pTextBxs = 
mrWrt.m_pTextBxs.get() )
+    while (i > 0)
     {
+        --i;
+
         // "dummy char" (or any Count ?) - why? Only Microsoft knows it.
         GetStream().WriteChar( i );
 
@@ -2308,6 +2310,9 @@ SwEscherEx::SwEscherEx(SvStream* pStrm, WW8Export& 
rWW8Wrt)
             CloseContainer();   // ESCHER_SpContainer
         }
         CloseContainer();   // ESCHER_DgContainer
+
+        pSdrObjs = mrWrt.m_pSdrObjs.get();
+        m_pTextBxs = mrWrt.m_pTextBxs.get();
     }
 }