tdf#114427 improve fix, wedo need to free it regardless

Change-Id: I7c35bcd839d7b7b2e8a9ce83f732e14b68ca58eb
Reviewed-on: https://gerrit.libreoffice.org/59172
Tested-by: Xisco Faulí <xiscofauli@libreoffice.org>
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

1 files changed, 3 insertions, 6 deletions

diff --git a/svx/source/unodraw/unoshape.cxx b/svx/source/unodraw/unoshape.cxx
index aea443acb6fa..a61ce3fbc79d 100644
--- a/svx/source/unodraw/unoshape.cxx
+++ b/svx/source/unodraw/unoshape.cxx
@@ -1357,11 +1357,13 @@ void SAL_CALL SvxShape::dispose()
     mpImpl->maDisposeListeners.disposeAndClear(aEvt);
     mpImpl->maPropertyNotifier.disposing();
 
-    bool bFreeSdrObject = false;
     if ( HasSdrObject() )
     {
         SdrObject* pObject = GetSdrObject();
+
         EndListening( pObject->getSdrModelFromSdrObject() );
+        bool bFreeSdrObject = false;
+
         if ( pObject->IsInserted() && pObject->getSdrPageFromSdrObject() )
         {
             OSL_ENSURE( HasSdrObjectOwnership(), "SvxShape::dispose: is the below code correct?" );
@@ -1381,11 +1383,6 @@ void SAL_CALL SvxShape::dispose()
                 }
             }
         }
-    }
-
-    if ( HasSdrObject() ) //tdf#114427 refetch SdrObject in light of RemoveObject
-    {
-        SdrObject* pObject = GetSdrObject();
 
         pObject->setUnoShape(nullptr);