[PATCH libreoffice-4-0] fixed out of bounds vector access

Eike Rathke (via Code Review) gerrit at gerrit.libreoffice.org
Tue Apr 23 09:36:42 PDT 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/3584

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/84/3584/1

fixed out of bounds vector access

in ScColumn::FindNextVisibleRowWithContent() if starting from the
position all including the last present cell of a column are blank

Change-Id: I0270331444f1707b6ad413f6127ae4c2cf0b1984
(cherry picked from commit 3874bad70ea48deed91b9966b3d35782b5584f7d)
---
M sc/source/core/data/column2.cxx
1 file changed, 1 insertion(+), 1 deletion(-)



diff --git a/sc/source/core/data/column2.cxx b/sc/source/core/data/column2.cxx
index 2bdb401..c782f89 100644
--- a/sc/source/core/data/column2.cxx
+++ b/sc/source/core/data/column2.cxx
@@ -1347,7 +1347,7 @@
             bool bThere = Search( nRow, nIndex );
             if( bThere && !maItems[nIndex].pCell->IsBlank())
                 return nRow;
-            else if(nIndex >= maItems.size())
+            else if((bThere ? nIndex+1 : nIndex) >= maItems.size())
                 return MAXROW;
             else
             {

-- 
To view, visit https://gerrit.libreoffice.org/3584
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I0270331444f1707b6ad413f6127ae4c2cf0b1984
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Eike Rathke <erack at redhat.com>



More information about the LibreOffice mailing list