Avoid crash when using inset-select-all in tabular

When changing cell idx, it is important to set the pit to a reasonable
value.

(cherry picked from commit a69e01d0b47aebbdfc3139533fcb7f1d0c04f156)

diff --git a/src/BufferView.cpp b/src/BufferView.cpp
index 66278e09854a85cd315b48cb21fa1b77ab75f251..6b3a528f483a164fdeaea66e2e3594421a37c189 100644 (file)
--- a/src/BufferView.cpp
+++ b/src/BufferView.cpp
@@ -1889,10 +1889,12 @@ void BufferView::dispatch(FuncRequest const & cmd, DispatchResult & dr)
                        // At least one complete cell is selected and inset is a table.
                        // Select all cells
                        cur.idx() = 0;
+                       cur.pit() = 0;
                        cur.pos() = 0;
                        cur.resetAnchor();
                        cur.selection(true);
                        cur.idx() = cur.lastidx();
+                       cur.pit() = cur.lastpit();
                        cur.pos() = cur.lastpos();
                } else {
                        // select current cell

diff --git a/status.23x b/status.23x
index baee939f05417873c42b052dc5fe00cb856f239d..b8e5306df070d6f94a6cdf662095d1be3b767bef 100644 (file)
--- a/status.23x
+++ b/status.23x
@@ -117,6 +117,8 @@ What's new
 
 - Fix cases where text is offset vertically (bug 10532, 10767).
 
+- Fix crash when using inset-select-all in tabular.
+
 
 * INTERNALS