diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2022-02-23 10:03:49 +0000 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2022-02-23 12:15:37 +0100 |
| commit | b2965afb8ba8ac155206a8dd790cd54ecfbf6d33 (patch) | |
| tree | 01f672bde4a392ef121e2063e261c2c08b58b2a0 | |
| parent | b1fe159fa39b2fc5217848e14d68c674979c181c (diff) | |
cid#1500665 Dereference before null check
Change-Id: I451c0faad1060ff7e97ef24af61940f402e1b243
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/130416
Tested-by: Caolán McNamara <caolanm@redhat.com>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
| -rw-r--r-- | chart2/source/tools/ConfigColorScheme.cxx | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/chart2/source/tools/ConfigColorScheme.cxx b/chart2/source/tools/ConfigColorScheme.cxx index 50d72a88207e..acb42fcfb729 100644 --- a/chart2/source/tools/ConfigColorScheme.cxx +++ b/chart2/source/tools/ConfigColorScheme.cxx @@ -124,9 +124,7 @@ void ConfigColorScheme::retrieveConfigColors() new impl::ChartConfigItem( *this )); m_apChartConfigItem->addPropertyNotification( aSeriesPropName ); } - OSL_ASSERT(m_apChartConfigItem); - if (!m_apChartConfigItem) - return; + assert(m_apChartConfigItem && "this can only be set at this point"); // retrieve colors uno::Any aValue( |