tdf#108889 ref count goes negative with 0 undo levels configured

Change-Id: Ic28c3738f951af6e5371f88b4bb061a0aadf16e1
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/151332
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>

1 files changed, 6 insertions, 9 deletions

diff --git a/svx/source/svdraw/svdedtv.cxx b/svx/source/svdraw/svdedtv.cxx
index 96e61779e04b..27015e410e9d 100644
--- a/svx/source/svdraw/svdedtv.cxx
+++ b/svx/source/svdraw/svdedtv.cxx
@@ -712,9 +712,9 @@ void SdrEditView::ForceMarkedObjToAnotherPage()
     }
 }
 
-std::vector<SdrObject*> SdrEditView::DeleteMarkedList(SdrMarkList const& rMark)
+std::vector<rtl::Reference<SdrObject>> SdrEditView::DeleteMarkedList(SdrMarkList const& rMark)
 {
-    std::vector<SdrObject*> ret;
+    std::vector<rtl::Reference<SdrObject>> ret;
     if (rMark.GetMarkCount()!=0)
     {
         rMark.ForceSort();
@@ -761,13 +761,13 @@ std::vector<SdrObject*> SdrEditView::DeleteMarkedList(SdrMarkList const& rMark)
                     aUpdaters.push_back(new E3DModifySceneSnapRectUpdater(pObj));
                 }
 
-                pOL->RemoveObject(nOrdNum);
-
                 if( !bUndo )
                 {
-                    // tdf#108863 don't delete objects before EndUndo()
+                    // tdf#108863 and tdf#108889 don't delete objects before EndUndo()
                     ret.push_back(pObj);
                 }
+
+                pOL->RemoveObject(nOrdNum);
             }
 
             // fire scene updaters
@@ -865,10 +865,7 @@ void SdrEditView::DeleteMarkedObj()
         // original stuff: remove selected objects. Handle clear will
         // do something only once
         auto temp(DeleteMarkedList(GetMarkedObjectList()));
-        for (auto p : temp)
-        {
-            lazyDeleteObjects.push_back(p);
-        }
+        lazyDeleteObjects.insert(lazyDeleteObjects.end(), temp.begin(), temp.end());
         GetMarkedObjectListWriteAccess().Clear();
         maHdlList.Clear();