[Libreoffice-commits] core.git: sc/source

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


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

New commits:
commit 5779b4966954d845f3c526c340bdee8c6a92ef95
Author:     Caolán McNamara <caolanm at redhat.com>
AuthorDate: Wed Aug 15 10:42:33 2018 +0100
Commit:     Caolán McNamara <caolanm at redhat.com>
CommitDate: Wed Aug 15 13:54:08 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/59060
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/sc/source/ui/unoobj/PivotTableDataProvider.cxx b/sc/source/ui/unoobj/PivotTableDataProvider.cxx
index abbe7e2265fd..94bf491c2e13 100644
--- a/sc/source/ui/unoobj/PivotTableDataProvider.cxx
+++ b/sc/source/ui/unoobj/PivotTableDataProvider.cxx
@@ -618,7 +618,7 @@ PivotTableDataProvider::assignLabelsToDataSequence(size_t nIndex)
     {
         aLabel = ScResId(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