diff options
author | Caolán McNamara <caolanm@redhat.com> | 2014-05-29 10:45:39 +0100 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2014-05-29 11:03:09 +0100 |
commit | 24ecc003382ef1c3eac60db4c0536dfd23b0315a (patch) | |
tree | 4231ca47582b5c4fa241d68b749f77ed465a3230 | |
parent | 56958bd0d9a150ee5230c02c03b5756500bc64b4 (diff) |
Resolves: fdo#79392 don't access after last row, when last row is max last row
Change-Id: I06749af90bed6a43f3fb6b3b5ceb61d5fbbc7c02
(cherry picked from commit a35d92197d850527bd294eb6197b5bf83109d5be)
-rw-r--r-- | sc/source/core/data/column3.cxx | 23 |
1 files changed, 13 insertions, 10 deletions
diff --git a/sc/source/core/data/column3.cxx b/sc/source/core/data/column3.cxx index d023b162bb2b..aa84d8dce328 100644 --- a/sc/source/core/data/column3.cxx +++ b/sc/source/core/data/column3.cxx @@ -182,19 +182,22 @@ void ScColumn::DeleteRow( SCROW nStartRow, SCSIZE nSize ) // Check if there are any cells below the end row that will get shifted. bool bShiftCells = false; - aPos = maCells.position(itCell, nEndRow+1); - itCell = aPos.first; - if (itCell->type == sc::element_type_empty) + if (nEndRow < MAXROWCOUNT-1) //only makes sense to do this if there *is* a row after the end row { - // This block is empty. See if there is any block that follows. - sc::CellStoreType::iterator itTest = itCell; - ++itTest; - if (itTest != maCells.end()) - // Non-empty block follows -> cells that will get shifted. + aPos = maCells.position(itCell, nEndRow+1); + itCell = aPos.first; + if (itCell->type == sc::element_type_empty) + { + // This block is empty. See if there is any block that follows. + sc::CellStoreType::iterator itTest = itCell; + ++itTest; + if (itTest != maCells.end()) + // Non-empty block follows -> cells that will get shifted. + bShiftCells = true; + } + else bShiftCells = true; } - else - bShiftCells = true; sc::SingleColumnSpanSet aNonEmptySpans; if (bShiftCells) |