diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2018-10-19 17:32:06 +0100 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2018-10-20 20:28:03 +0200 |
| commit | a8a0d0fa151aee294fc3744c6fbc0729b2a7bcee (patch) | |
| tree | 0bc426849735e8bc7e14f95d1d30e4275cab931b /winaccessibility/source | |
| parent | 5d0b961b1962474fd5c5be9a6a064114a710bc1b (diff) | |
pvs-studio: pointer was utilized before it was verified against nullptr
Change-Id: I601d30f8222f7c666eab4727c84e612484e01eed
Reviewed-on: https://gerrit.libreoffice.org/62067
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'winaccessibility/source')
| -rw-r--r-- | winaccessibility/source/service/AccDescendantManagerEventListener.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/winaccessibility/source/service/AccDescendantManagerEventListener.cxx b/winaccessibility/source/service/AccDescendantManagerEventListener.cxx index 3f865cd2e27d..c8fc771ae503 100644 --- a/winaccessibility/source/service/AccDescendantManagerEventListener.cxx +++ b/winaccessibility/source/service/AccDescendantManagerEventListener.cxx @@ -183,7 +183,7 @@ bool AccDescendantManagerEventListener::NotifyChildEvent(short nWinEvent,const A XAccessible* pAcc = xChild.get(); pAgent->NotifyAccEvent(nWinEvent, pAcc); - if (pAgent && pAgent->IsStateManageDescendant(m_xAccessible.get()) + if (pAgent->IsStateManageDescendant(m_xAccessible.get()) && (nWinEvent == UM_EVENT_SELECTION_CHANGED_REMOVE)) { pAgent->DeleteAccObj( pAcc ); |