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

Libreoffice Gerrit user logerrit at kemper.freedesktop.org
Wed Aug 15 12:11:01 UTC 2018


 sc/source/ui/unoobj/PivotTableDataProvider.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit ff16f75a474533187253a190557d6e2195a7e4cb
Author:     Caolán McNamara <caolanm at redhat.com>
AuthorDate: Wed Aug 15 10:42:33 2018 +0100
Commit:     Markus Mohrhard <markus.mohrhard at googlemail.com>
CommitDate: Wed Aug 15 14:10:37 2018 +0200

    tdf#115086 crash in assignLabelsToDataSequence
    
    since...
    
    commit c45358e928b9b9419bba10a0fcb661696cb7647a
    Date:   Mon Apr 24 00:09:08 2017 +0200
    
        preserve "Total" label on round-trip, remove code duplication
    
    Change-Id: I83ed20e1cab8d67f2f5aedcaf862756d9548bf9f
    Reviewed-on: https://gerrit.libreoffice.org/59062
    Tested-by: Jenkins
    Reviewed-by: Markus Mohrhard <markus.mohrhard at googlemail.com>

diff --git a/sc/source/ui/unoobj/PivotTableDataProvider.cxx b/sc/source/ui/unoobj/PivotTableDataProvider.cxx
index cfdea1bed39e..12f432a19085 100644
--- a/sc/source/ui/unoobj/PivotTableDataProvider.cxx
+++ b/sc/source/ui/unoobj/PivotTableDataProvider.cxx
@@ -617,7 +617,7 @@ PivotTableDataProvider::assignLabelsToDataSequence(size_t nIndex)
     {
         aLabel = ScGlobal::GetRscString(STR_PIVOT_TOTAL);
     }
-    else
+    else if (nIndex < m_aLabels.size())
     {
         for (ValueAndFormat const & rItem : m_aLabels[nIndex])
         {


More information about the Libreoffice-commits mailing list