diff options
author | Caolán McNamara <caolanm@redhat.com> | 2014-02-27 09:55:18 +0000 |
---|---|---|
committer | Tor Lillqvist <tml@collabora.com> | 2014-03-07 08:37:03 +0200 |
commit | eb66d534b78c471ebbffe960d7df5a683a7e8f8e (patch) | |
tree | 5e48d20dce529c5f9eff9008cd736421fda3b7c3 /extensions | |
parent | 5b4f4ad23d6c7369fb587b16076e78c74b609bd7 (diff) |
coverity#704633 Dereference after null check
(cherry picked from commit ccf0f7ef03bf98831a3e21b4b1327673a590b918)
Change-Id: I4985249110da93ba67881dcd1c8cd1dc66ae08e2
Reviewed-on: https://gerrit.libreoffice.org/8379
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Tested-by: Miklos Vajna <vmiklos@collabora.co.uk>
Diffstat (limited to 'extensions')
-rw-r--r-- | extensions/source/propctrlr/browserlistbox.cxx | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/extensions/source/propctrlr/browserlistbox.cxx b/extensions/source/propctrlr/browserlistbox.cxx index fa6bfcc36aca..9a8cf3718070 100644 --- a/extensions/source/propctrlr/browserlistbox.cxx +++ b/extensions/source/propctrlr/browserlistbox.cxx @@ -1196,11 +1196,12 @@ namespace pcr // So, we manually switch this to read-only. if ( xControl.is() && ( xControl->getControlType() == PropertyControlType::Unknown ) ) { - Edit* pControlWindowAsEdit = dynamic_cast< Edit* >( rLine.pLine->getControlWindow() ); - if ( pControlWindowAsEdit ) - pControlWindowAsEdit->SetReadOnly( sal_True ); + Window *pWindow = rLine.pLine->getControlWindow(); + Edit* pControlWindowAsEdit = dynamic_cast<Edit*>(pWindow); + if (pControlWindowAsEdit) + pControlWindowAsEdit->SetReadOnly(sal_True); else - pControlWindowAsEdit->Enable( sal_False ); + pWindow->Enable(sal_False); } } } |