coverity#1397056 Dereference null return value

Change-Id: Ieda3afdbdbcc8aa21f6584edc21738c4edb84000

coverity#1397056 Dereference null return value
Change-Id: Ieda3afdbdbcc8aa21f6584edc21738c4edb84000
221a3413 · Caolán McNamara · 3249009b · 221a3413
Kaydet (Commit) 221a3413 authored Ara 12, 2016 tarafından Caolán McNamara
Hide whitespace changes
Inline Side-by-side

Showing with 9 additions and 6 deletions

treelistbox.cxx svtools/source/contnr/treelistbox.cxx +9 -6

No files found.
--- a/svtools/source/contnr/treelistbox.cxx
+++ b/svtools/source/contnr/treelistbox.cxx
@@ -1844,12 +1844,15 @@ void SvTreeListBox::SetCheckButtonState( SvTreeListEntry* pEntry, SvButtonState

 void SvTreeListBox::SetCheckButtonInvisible( SvTreeListEntry* pEntry)
 {
-    if( nTreeFlags & SvTreeFlags::CHKBTN )
-    {
-        SvLBoxButton* pItem = static_cast<SvLBoxButton*>(pEntry->GetFirstItem(SvLBoxItemType::Button));
-        pItem->SetStateInvisible();
-        InvalidateEntry( pEntry );
-    }
+    SvLBoxButton* pItem = (nTreeFlags & SvTreeFlags::CHKBTN) ?
+        static_cast<SvLBoxButton*>(pEntry->GetFirstItem(SvLBoxItemType::Button)) :
+        nullptr;
+
+    if (!pItem)
+        return;
+
+    pItem->SetStateInvisible();
+    InvalidateEntry(pEntry);
 }

 SvButtonState SvTreeListBox::GetCheckButtonState( SvTreeListEntry* pEntry ) const