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

Markus Mohrhard markus.mohrhard at googlemail.com
Fri Dec 22 08:01:30 UTC 2017


 chart2/source/controller/sidebar/ChartElementsPanel.cxx |    6 ++++++
 1 file changed, 6 insertions(+)

New commits:
commit 1d2eb0b64339a771152bb17831c83373830e5c38
Author: Markus Mohrhard <markus.mohrhard at googlemail.com>
Date:   Thu Dec 21 23:42:19 2017 +0100

    prevent crash when trying to edit broken chart documents
    
    Change-Id: Ie5fb706011ca600b42e5e2d3770e919b56347edd
    Reviewed-on: https://gerrit.libreoffice.org/46938
    Reviewed-by: Markus Mohrhard <markus.mohrhard at googlemail.com>
    Tested-by: Jenkins <ci at libreoffice.org>

diff --git a/chart2/source/controller/sidebar/ChartElementsPanel.cxx b/chart2/source/controller/sidebar/ChartElementsPanel.cxx
index 7f2a471ce0f7..2f9d7ebaec27 100644
--- a/chart2/source/controller/sidebar/ChartElementsPanel.cxx
+++ b/chart2/source/controller/sidebar/ChartElementsPanel.cxx
@@ -397,10 +397,16 @@ css::uno::Reference<css::chart2::XChartType> getChartType(const css::uno::Refere
 
     css::uno::Sequence<css::uno::Reference<css::chart2::XCoordinateSystem>> xCooSysSequence(xCooSysContainer->getCoordinateSystems());
 
+    if (xCooSysSequence.getLength() == 0)
+        return css::uno::Reference<css::chart2::XChartType>();
+
     css::uno::Reference<css::chart2::XChartTypeContainer> xChartTypeContainer(xCooSysSequence[0], css::uno::UNO_QUERY_THROW);
 
     css::uno::Sequence<css::uno::Reference<css::chart2::XChartType>> xChartTypeSequence(xChartTypeContainer->getChartTypes());
 
+    if (xChartTypeSequence.getLength() == 0)
+        return css::uno::Reference<css::chart2::XChartType>();
+
     return xChartTypeSequence[0];
 }
 


More information about the Libreoffice-commits mailing list