coverity#1130356 Unchecked dynamic_cast

Change-Id: I1481729063c20b75841372363ab936154c989b3f

1 files changed, 6 insertions, 3 deletions

diff --git a/sw/source/core/frmedt/fefly1.cxx b/sw/source/core/frmedt/fefly1.cxx
index a149de874b43..cbcdbcfadaf2 100644
--- a/sw/source/core/frmedt/fefly1.cxx
+++ b/sw/source/core/frmedt/fefly1.cxx
@@ -794,7 +794,10 @@ void SwFEShell::Insert( const OUString& rGrfName, const OUString& rFltName,
     StartAllAction();
     SwShellCrsr *pStartCursor = dynamic_cast<SwShellCrsr*>(this->GetSwCrsr());
     SwShellCrsr *pCursor = pStartCursor;
-    do {
+    do
+    {
+        if (!pCursor)
+            break;
 
         // Has the anchor not been set or been set incompletely?
         if( pFlyAttrSet )
@@ -839,8 +842,8 @@ void SwFEShell::Insert( const OUString& rGrfName, const OUString& rFltName,
                                 pGrfAttrSet, pFrmFmt );
         OSL_ENSURE( pFmt, "Doc->Insert(notxt) failed." );
 
-    } while( (pCursor = dynamic_cast<SwShellCrsr*>(pCursor->GetNext()))
-             != pStartCursor );
+        pCursor = dynamic_cast<SwShellCrsr*>(pCursor->GetNext());
+    } while( pCursor != pStartCursor );
 
     EndAllAction();