[Libreoffice-commits] core.git: Branch 'libreoffice-6-0' - svx/source

[Julien Nabet]

 svx/source/fmcomp/fmgridcl.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 1823d8d0f1a1f29db8cb4bfbfbf2b7b2c94dcbb7
Author: Julien Nabet <serval2412 at yahoo.fr>
Date:   Sun Dec 31 15:50:36 2017 +0100

    tdf#114775: fix crash when deleting some rows
    
    see bt https://bugs.documentfoundation.org/attachment.cgi?id=138757
    
    Change-Id: I493bf47b1541262a5ea78b541bc1f47f93cc73df
    Reviewed-on: https://gerrit.libreoffice.org/47227
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Julien Nabet <serval2412 at yahoo.fr>

diff --git a/svx/source/fmcomp/fmgridcl.cxx b/svx/source/fmcomp/fmgridcl.cxx
index 57af471fda3f..0ed000716416 100644
--- a/svx/source/fmcomp/fmgridcl.cxx
+++ b/svx/source/fmcomp/fmgridcl.cxx
@@ -1840,7 +1840,7 @@ Sequence< Any> FmGridControl::getSelectionBookmarks()
         // change our selection. So a "FirstSelected(); SeekCursor(); NextSelected();" may produce unpredictable results.
         // That's why we _first_ collect the indices of the selected rows and _then_ their bookmarks.
         long nIdx = FirstSelectedRow();
-        while (nIdx >= 0)
+        while (nIdx != BROWSER_ENDOFSELECTION)
         {
             // (we misuse the bookmarks array for this ...)
             pBookmarks[i++] <<= (sal_Int32)nIdx;