coverity#736804 Dereference before null check

Change-Id: If4b656ccea684fda64a3f93f5b6729c713be48b0

1 files changed, 3 insertions, 5 deletions

diff --git a/sd/source/ui/func/fupoor.cxx b/sd/source/ui/func/fupoor.cxx
index 06e431d9fa83..76c59e158deb 100644
--- a/sd/source/ui/func/fupoor.cxx
+++ b/sd/source/ui/func/fupoor.cxx
@@ -802,7 +802,7 @@ bool FuPoor::KeyInput(const KeyEvent& rKEvt)
     // and feed character to object
     if(!bReturn && !mpDocSh->IsReadOnly())
     {
-        if(!mpView->IsTextEdit() && mpViewShell)
+        if (!mpView->IsTextEdit())
         {
             const SdrMarkList& rMarkList = mpView->GetMarkedObjectList();
 
@@ -836,11 +836,9 @@ bool FuPoor::KeyInput(const KeyEvent& rKEvt)
             {
                 // test if there is a title object there. If yes, try to
                 // set it to edit mode and start typing...
-                if( dynamic_cast< const DrawViewShell *>( mpViewShell ) !=  nullptr
-                    && EditEngine::IsSimpleCharInput(rKEvt))
+                DrawViewShell* pDrawViewShell = dynamic_cast<DrawViewShell*>(mpViewShell);
+                if (pDrawViewShell && EditEngine::IsSimpleCharInput(rKEvt))
                 {
-                    DrawViewShell* pDrawViewShell =
-                        static_cast<DrawViewShell*>(mpViewShell);
                     SdPage* pActualPage = pDrawViewShell->GetActualPage();
                     SdrTextObj* pCandidate = 0L;