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

[Caolán McNamara (via logerrit)]

 sc/source/filter/excel/xltoolbar.cxx |   13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

New commits:
commit ffe6eb528dca34b19fecf738bce647ccb8367379
Author:     Caolán McNamara <caolanm at redhat.com>
AuthorDate: Tue Mar 30 09:24:58 2021 +0100
Commit:     Caolán McNamara <caolanm at redhat.com>
CommitDate: Tue Mar 30 12:34:53 2021 +0200

    cid#1473732 Untrusted loop bound
    
    Change-Id: I0d0585f8d379fc0586ee76527f96e50c92953d13
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/113338
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/sc/source/filter/excel/xltoolbar.cxx b/sc/source/filter/excel/xltoolbar.cxx
index f94f96f50033..acf6d8339f20 100644
--- a/sc/source/filter/excel/xltoolbar.cxx
+++ b/sc/source/filter/excel/xltoolbar.cxx
@@ -111,14 +111,15 @@ bool ScCTB::Read( SvStream &rS )
             SAL_WARN("sc.filter", "ScCTB::Read more entries claimed than stream could contain");
             return false;
         }
-    }
 
-    for ( sal_Int16 index = 0; index < nIndexes; ++index )
-    {
-        ScTBC aTBC;
-        aTBC.Read( rS );
-        rTBC.push_back( aTBC );
+        for ( sal_Int16 index = 0; index < nIndexes; ++index )
+        {
+            ScTBC aTBC;
+            aTBC.Read( rS );
+            rTBC.push_back( aTBC );
+        }
     }
+
     return true;
 }