coverity#1158136 Unchecked dynamic_cast

Change-Id: I8d4e7c497430faacd0d8f1138b9af8b2f11127a0

coverity#1158136 Unchecked dynamic_cast
Change-Id: I8d4e7c497430faacd0d8f1138b9af8b2f11127a0
85344383 · Caolán McNamara · 4f4a21c8 · 85344383
Kaydet (Commit) 85344383 authored Ock 25, 2014 tarafından Caolán McNamara
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 3 deletions

dlg_ObjectProperties.cxx chart2/source/controller/dialogs/dlg_ObjectProperties.cxx +7 -3

No files found.
--- a/chart2/source/controller/dialogs/dlg_ObjectProperties.cxx
+++ b/chart2/source/controller/dialogs/dlg_ObjectProperties.cxx
@@ -189,7 +189,8 @@ void ObjectPropertiesDialogParameter::init( const uno::Reference< frame::XModel
                    if( m_bIsCrossingAxisIsCategoryAxis )
                    {
                        ChartModel* pModel = dynamic_cast<ChartModel*>(xChartModel.get());
-                        m_aCategories = DiagramHelper::getExplicitSimpleCategories( *pModel );
+                        if (pModel)
+                            m_aCategories = DiagramHelper::getExplicitSimpleCategories( *pModel );
                    }
                }
@@ -197,8 +198,11 @@ void ObjectPropertiesDialogParameter::init( const uno::Reference< frame::XModel
                if ( nDimensionIndex == 0 && aData.AxisType == chart2::AxisType::CATEGORY )
                {
                    ChartModel* pModel = dynamic_cast<ChartModel*>(xChartModel.get());
-                    ExplicitCategoriesProvider aExplicitCategoriesProvider( xCooSys, *pModel );
+                    if (pModel)
-                    m_bComplexCategoriesAxis = aExplicitCategoriesProvider.hasComplexCategories();
+                    {
+                        ExplicitCategoriesProvider aExplicitCategoriesProvider( xCooSys, *pModel );
+                        m_bComplexCategoriesAxis = aExplicitCategoriesProvider.hasComplexCategories();
+                    }
                }
            }
        }