cid#1474044 Untrusted loop bound

Change-Id: I9f060fe0b56905def7b81a9b7da0b504367eb36c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/113337
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>

1 files changed, 6 insertions, 6 deletions

diff --git a/basic/source/sbx/sbxarray.cxx b/basic/source/sbx/sbxarray.cxx
index fa2a2aa69f9a..4f5a9fd3cfb0 100644
--- a/basic/source/sbx/sbxarray.cxx
+++ b/basic/source/sbx/sbxarray.cxx
@@ -543,13 +543,13 @@ bool SbxDimArray::LoadData( SvStream& rStrm, sal_uInt16 nVer )
             SAL_WARN("basic", "SbxDimArray::LoadData more entries claimed than stream could contain");
             return false;
         }
-    }
 
-    for( short i = 0; i < nDimension && rStrm.GetError() == ERRCODE_NONE; i++ )
-    {
-        sal_Int16 lb(0), ub(0);
-        rStrm.ReadInt16( lb ).ReadInt16( ub );
-        AddDim( lb, ub );
+        for (short i = 0; i < nDimension && rStrm.GetError() == ERRCODE_NONE; ++i)
+        {
+            sal_Int16 lb(0), ub(0);
+            rStrm.ReadInt16( lb ).ReadInt16( ub );
+            AddDim( lb, ub );
+        }
     }
     return SbxArray::LoadData( rStrm, nVer );
 }