[Libreoffice-commits] core.git: Branch 'libreoffice-7-1' - sc/source

[Noel Grandin (via logerrit)]

 sc/source/filter/xml/xmlexprt.cxx |    5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

New commits:
commit f92ea2dfaacc63a7a1721652d68faefd34395847
Author:     Noel Grandin <noelgrandin at gmail.com>
AuthorDate: Sat Nov 28 08:43:54 2020 +0200
Commit:     Xisco Fauli <xiscofauli at libreoffice.org>
CommitDate: Sat Nov 28 15:51:49 2020 +0100

    tdf#138466 Crash when creating different scenarios
    
    apparently this is a regression from
        commit 4176beb7ef831152ce92ac3fa31314438635ec2c
        Author: Noel Grandin <noelgrandin at 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 at collabora.co.uk>
    (cherry picked from commit 52aebe6986bcec07071adb3f94d6c09fea147044)
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106747
    Reviewed-by: Xisco Fauli <xiscofauli at libreoffice.org>

diff --git a/sc/source/filter/xml/xmlexprt.cxx b/sc/source/filter/xml/xmlexprt.cxx
index 1f31e5d52c6d..6df4093cc8eb 100644
--- a/sc/source/filter/xml/xmlexprt.cxx
+++ b/sc/source/filter/xml/xmlexprt.cxx
@@ -869,8 +869,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))