coverity#704754 Dereference after null check

Change-Id: I3c70c9cfe84dc74836cf733e424cf7c2cd195c4f

1 files changed, 17 insertions, 17 deletions

diff --git a/sd/source/ui/func/fuparagr.cxx b/sd/source/ui/func/fuparagr.cxx
index 512bd7add43a..e4250eddaa58 100644
--- a/sd/source/ui/func/fuparagr.cxx
+++ b/sd/source/ui/func/fuparagr.cxx
@@ -91,28 +91,28 @@ void FuParagraph::DoExecute( SfxRequest& rReq )
 
         SdAbstractDialogFactory* pFact = SdAbstractDialogFactory::Create();
         SfxAbstractTabDialog* pDlg = pFact ? pFact->CreateSdParagraphTabDlg(NULL, &aNewAttr ) : 0;
-        if( pDlg )
+        if (!pDlg)
+            return;
+
+        sal_uInt16 nResult = pDlg->Execute();
+
+        switch( nResult )
         {
-            sal_uInt16 nResult = pDlg->Execute();
+            case RET_OK:
+            {
+                rReq.Done( *( pDlg->GetOutputItemSet() ) );
+
+                pArgs = rReq.GetArgs();
+            }
+            break;
 
-            switch( nResult )
+            default:
             {
-                case RET_OK:
-                {
-                    rReq.Done( *( pDlg->GetOutputItemSet() ) );
-
-                    pArgs = rReq.GetArgs();
-                }
-                break;
-
-                default:
-                {
-                    delete pDlg;
-                }
-                return; // Cancel
+                delete pDlg;
             }
-            delete( pDlg );
+            return; // Cancel
         }
+        delete( pDlg );
     }
     mpView->SetAttributes( *pArgs );