diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2019-04-01 00:25:16 +0300 |
|---|---|---|
| committer | Mike Kaganski <mike.kaganski@collabora.com> | 2019-04-01 00:49:12 +0200 |
| commit | 85456fae54029edd26df2277a9eec5e2fe3d9739 (patch) | |
| tree | fd55ad48b34c66f0b839edcbc952a8a5a0752c72 /winaccessibility | |
| parent | 1d35b89e7581e12024769d19bfb6d6eb2c3274b8 (diff) | |
tdf#120703 PVS: Silence V522 warnings
V522 There might be dereferencing of a potential null pointer.
Change-Id: Ie617b41a8f8d334022cf5313b242a236baedba48
Reviewed-on: https://gerrit.libreoffice.org/70017
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'winaccessibility')
| -rw-r--r-- | winaccessibility/source/service/AccFrameEventListener.cxx | 4 | ||||
| -rw-r--r-- | winaccessibility/source/service/AccObjectWinManager.cxx | 5 |
2 files changed, 4 insertions, 5 deletions
diff --git a/winaccessibility/source/service/AccFrameEventListener.cxx b/winaccessibility/source/service/AccFrameEventListener.cxx index 8f7a6bc32753..cf1968d65032 100644 --- a/winaccessibility/source/service/AccFrameEventListener.cxx +++ b/winaccessibility/source/service/AccFrameEventListener.cxx @@ -85,8 +85,8 @@ void AccFrameEventListener::HandleChildChangedEvent(Any oldValue, Any newValue) { XAccessible* pAcc = xChild.get(); - VCLXWindow* pvclwindow = - dynamic_cast<VCLXWindow*>(m_xAccessible.get()); + assert(dynamic_cast<VCLXWindow*>(m_xAccessible.get())); + VCLXWindow* pvclwindow = static_cast<VCLXWindow*>(m_xAccessible.get()); const SystemEnvData* systemdata = pvclwindow->GetWindow()->GetSystemData(); diff --git a/winaccessibility/source/service/AccObjectWinManager.cxx b/winaccessibility/source/service/AccObjectWinManager.cxx index 90714dea6339..72774cf42d37 100644 --- a/winaccessibility/source/service/AccObjectWinManager.cxx +++ b/winaccessibility/source/service/AccObjectWinManager.cxx @@ -111,9 +111,8 @@ AccObjectWinManager::Get_ToATInterface(HWND hWnd, long lParam, WPARAM wParam) if ( pRetIMAcc && lParam == OBJID_CLIENT ) { - IAccessible* pTemp = dynamic_cast<IAccessible*>( pRetIMAcc ); - LRESULT result = LresultFromObject(IID_IAccessible, wParam, pTemp); - pTemp->Release(); + LRESULT result = LresultFromObject(IID_IAccessible, wParam, pRetIMAcc); + pRetIMAcc->Release(); return result; } return 0; |