cppCheck: Possible NULL pointer dereference

Change-Id: I1d740eea5c26e0515835c4dc6199a387b6fa8f6b
Signed-off-by: jailletc36 <christophe.jaillet@wanadoo.fr>
Reviewed-on: https://gerrit.libreoffice.org/1154
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Eike Rathke <erack@redhat.com>

1 files changed, 8 insertions, 6 deletions

diff --git a/sw/source/core/frmedt/fefly1.cxx b/sw/source/core/frmedt/fefly1.cxx
index 2673b314815e..50ef8d7ad4e2 100644
--- a/sw/source/core/frmedt/fefly1.cxx
+++ b/sw/source/core/frmedt/fefly1.cxx
@@ -841,14 +841,16 @@ void SwFEShell::Insert( const String& rGrfName, const String& rFltName,
         const Point aPt( GetCrsrDocPos() );
         SwFlyFrm* pFrm = pFmt->GetFrm( &aPt );
 
-        // fdo#36681: Invalidate the content and layout to refresh the picture anchoring
-        // properly
-        SwPageFrm* pPageFrm = pFrm->FindPageFrmOfAnchor();
-        pPageFrm->InvalidateFlyLayout();
-        pPageFrm->InvalidateCntnt();
-
         if( pFrm )
+        {
+            // fdo#36681: Invalidate the content and layout to refresh
+            // the picture anchoring properly
+            SwPageFrm* pPageFrm = pFrm->FindPageFrmOfAnchor();
+            pPageFrm->InvalidateFlyLayout();
+            pPageFrm->InvalidateCntnt();
+
             SelectFlyFrm( *pFrm, sal_True );
+        }
         else
             GetLayout()->SetAssertFlyPages();
     }