sc/source/core/data/documen4.cxx | 2 ++ 1 file changed, 2 insertions(+)
New commits: commit 0e7593ed856ac49c1d69407b8d0db5b771cc2805 Author: Eike Rathke <er...@redhat.com> Date: Thu Sep 14 17:45:22 2017 +0200 InsertMatrixFormula,InsertTableOp: assert valid cols/rows We don't check and return without operation for these, so the caller is responsible for handling bad prerequisites. Change-Id: I47bf5dc171c2362bf7096022058a51460c6e50e0 diff --git a/sc/source/core/data/documen4.cxx b/sc/source/core/data/documen4.cxx index ae608b3c2e94..d1eebdbe3cfe 100644 --- a/sc/source/core/data/documen4.cxx +++ b/sc/source/core/data/documen4.cxx @@ -271,6 +271,7 @@ void ScDocument::InsertMatrixFormula(SCCOL nCol1, SCROW nRow1, SAL_WARN("sc", "ScDocument::InsertMatrixFormula: No table marked"); return; } + assert( ValidColRow( nCol1, nRow1) && ValidColRow( nCol2, nRow2)); SCTAB nTab1 = *rMark.begin(); @@ -353,6 +354,7 @@ void ScDocument::InsertTableOp(const ScTabOpParam& rParam, // multiple (repeate { PutInOrder(nCol1, nCol2); PutInOrder(nRow1, nRow2); + assert( ValidColRow( nCol1, nRow1) && ValidColRow( nCol2, nRow2)); SCTAB i, nTab1; SCCOL j; SCROW k; _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits