[Libreoffice-commits] .: sc/source

Libreoffice Gerrit user logerrit at kemper.freedesktop.org
Wed Sep 26 11:05:41 PDT 2012


 sc/source/core/data/dociter.cxx |    6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

New commits:
commit 70412d0f3f7979d3d069be8121d49dbb96775cce
Author: Markus Mohrhard <markus.mohrhard at googlemail.com>
Date:   Wed Sep 26 20:01:20 2012 +0200

    don't access out of bound array index, fdo#55022
    
    Change-Id: Ica088822055b2fb5bbd6dabe6482b7cb6c472c48

diff --git a/sc/source/core/data/dociter.cxx b/sc/source/core/data/dociter.cxx
index d6d59ce..2d8808e 100644
--- a/sc/source/core/data/dociter.cxx
+++ b/sc/source/core/data/dociter.cxx
@@ -565,7 +565,11 @@ bool ScDBQueryDataIterator::DataAccessInternal::getCurrent(Value& rValue)
 
         SCROW nThisRow = ScDBQueryDataIterator::GetRowByColEntryIndex(*mpDoc, nTab, nCol, nColRow);
         while ( (nColRow < nCellCount) && (nThisRow < nRow) )
-            nThisRow = ScDBQueryDataIterator::GetRowByColEntryIndex(*mpDoc, nTab, nCol, ++nColRow);
+        {
+            ++nColRow;
+            if(nColRow < nCellCount)
+                nThisRow = ScDBQueryDataIterator::GetRowByColEntryIndex(*mpDoc, nTab, nCol, nColRow);
+        }
 
         if ( nColRow < nCellCount && nThisRow <= mpParam->nRow2 )
         {


More information about the Libreoffice-commits mailing list