diff options
author | Caolán McNamara <caolanm@redhat.com> | 2018-03-19 13:54:54 +0000 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2018-03-19 16:59:13 +0100 |
commit | a80d9d61b8dc98dc188c45a3d5d966c6d0ac2b79 (patch) | |
tree | ea1d4d09dbdb58a93a5de91b5cc6a1b40fee0152 /sc | |
parent | 9078fbd14be7c2e4f7c830286cceb0003a345472 (diff) |
coverity#1430075 Dereference after null check
Change-Id: Id4c9cc1463ba8b38a7b115d574fec8c9b1d55384
Reviewed-on: https://gerrit.libreoffice.org/51556
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'sc')
-rw-r--r-- | sc/source/ui/unoobj/docuno.cxx | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/sc/source/ui/unoobj/docuno.cxx b/sc/source/ui/unoobj/docuno.cxx index 6eeeef3c9684..88e1a43dd7b8 100644 --- a/sc/source/ui/unoobj/docuno.cxx +++ b/sc/source/ui/unoobj/docuno.cxx @@ -557,12 +557,14 @@ VclPtr<vcl::Window> ScModelObj::getDocWindow() ScViewData* pViewData = ScDocShell::GetViewData(); VclPtr<vcl::Window> pWindow; if (pViewData) + { pWindow = pViewData->GetActiveWin(); - LokChartHelper aChartHelper(pViewData->GetViewShell()); - vcl::Window* pChartWindow = aChartHelper.GetWindow(); - if (pChartWindow) - pWindow = pChartWindow; + LokChartHelper aChartHelper(pViewData->GetViewShell()); + vcl::Window* pChartWindow = aChartHelper.GetWindow(); + if (pChartWindow) + pWindow = pChartWindow; + } return pWindow; } |