diff options
| author | Noel Grandin <noelgrandin@gmail.com> | 2020-11-28 08:43:54 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-11-28 09:03:18 +0100 |
| commit | 52aebe6986bcec07071adb3f94d6c09fea147044 (patch) | |
| tree | 36a4e6f44b4f18c66b73747324b24343bcb188cc /sc | |
| parent | aabdd3a4393df9d6a354783d8c981ce9ca1b1a6c (diff) | |
tdf#138466 Crash when creating different scenarios
apparently this is a regression from
commit 4176beb7ef831152ce92ac3fa31314438635ec2c
Author: Noel Grandin <noelgrandin@gmail.com>
Date: Sat Sep 12 18:21:44 2020 +0200
tdf#133327 fix calc loading background color with many cols
but I have no idea how or why, so just fix the crash.
Change-Id: Ia2f056fc53527338c431d3b34bb64e6eedc8e955
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106799
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sc')
| -rw-r--r-- | sc/source/filter/xml/xmlexprt.cxx | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/sc/source/filter/xml/xmlexprt.cxx b/sc/source/filter/xml/xmlexprt.cxx index da8f7ce9c38a..dcaddbca0830 100644 --- a/sc/source/filter/xml/xmlexprt.cxx +++ b/sc/source/filter/xml/xmlexprt.cxx @@ -868,8 +868,9 @@ void ScXMLExport::ExportColumns(const sal_Int32 nTable, const ScRange& aColumnHe nColsRepeated = 1; } } - assert(nPrevIndex >= 0 && "coverity#1438402"); - WriteColumn(nPrevColumn, nColsRepeated, nPrevIndex, bPrevIsVisible); + // tdf#138466 + if (nPrevIndex != -1) + WriteColumn(nPrevColumn, nColsRepeated, nPrevIndex, bPrevIsVisible); if (!bIsClosed) CloseHeaderColumn(); if (pGroupColumns->IsGroupEnd(nColumn - 1)) |