vcl/source/filter/ipict/ipict.cxx |    8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

New commits:
commit 51e260ca3841d55213b96fc5816904b912f637f0
Author:     Caolán McNamara <caolan.mcnam...@collabora.com>
AuthorDate: Fri Aug 1 19:34:56 2025 +0100
Commit:     Caolán McNamara <caolan.mcnam...@collabora.com>
CommitDate: Fri Aug 1 22:03:17 2025 +0200

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

diff --git a/vcl/source/filter/ipict/ipict.cxx 
b/vcl/source/filter/ipict/ipict.cxx
index 98cb3ba331d6..cf39ebbb2a08 100644
--- a/vcl/source/filter/ipict/ipict.cxx
+++ b/vcl/source/filter/ipict/ipict.cxx
@@ -1104,10 +1104,11 @@ sal_uInt64 PictReader::ReadPixMapEtc( BitmapEx 
&rBitmap, bool bBaseAddr, bool bC
                                 nCount = aScanline.size() - i;
                             if (pPict->remainingSize() < nCount)
                                 return 0xffffffff;
-                            while( nCount-- )
+                            while (nCount > 0)
                             {
                                 pPict->ReadUChar( nDat );
                                 aScanline[ i++ ] = nDat;
+                                --nCount;
                             }
                         }
                         else
@@ -1118,8 +1119,11 @@ sal_uInt64 PictReader::ReadPixMapEtc( BitmapEx &rBitmap, 
bool bBaseAddr, bool bC
                             if (( i + nCount) > aScanline.size())
                                 nCount = aScanline.size() - i;
                             pPict->ReadUChar( nDat );
-                            while( nCount-- )
+                            while (nCount > 0)
+                            {
                                 aScanline[ i++ ] = nDat;
+                                --nCount;
+                            }
                         }
                     }
                     sal_uInt8* pTmp = aScanline.data();

Reply via email to