svx/source/svdraw/svdedtv1.cxx |   20 +++++++++++++++-----
 1 file changed, 15 insertions(+), 5 deletions(-)

New commits:
commit 0c8b5e055fefcd954cdc5dba64f45081bbc37d12
Author:     Sahil Gautam <sahil.gautam.ext...@allotropia.de>
AuthorDate: Sun Mar 16 01:35:17 2025 +0530
Commit:     Xisco Fauli <xiscofa...@libreoffice.org>
CommitDate: Mon Mar 17 14:55:19 2025 +0100

    dereference the pointer iff it's not null
    
    Change-Id: I1dc1a1ef7e03a86292478a39cfd6e145f3eb5630
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/182973
    Reviewed-by: Miklos Vajna <vmik...@collabora.com>
    Tested-by: Jenkins
    (cherry picked from commit 6e8a094a1a1d63ed2684900c5e0c4292f8d833a8)
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/183026
    Reviewed-by: Xisco Fauli <xiscofa...@libreoffice.org>

diff --git a/svx/source/svdraw/svdedtv1.cxx b/svx/source/svdraw/svdedtv1.cxx
index acf0b6885028..6cd843a78004 100644
--- a/svx/source/svdraw/svdedtv1.cxx
+++ b/svx/source/svdraw/svdedtv1.cxx
@@ -22,6 +22,7 @@
 #include <editeng/eeitem.hxx>
 #include <math.h>
 #include <svl/itemiter.hxx>
+#include <sal/log.hxx>
 #include <svl/whiter.hxx>
 #include <tools/bigint.hxx>
 #include <vcl/svapp.hxx>
@@ -218,11 +219,20 @@ void SdrEditView::MoveMarkedObj(const Size& rSiz, bool 
bCopy)
 
 bool SdrEditView::IsMarkedObjSizeValid(Size& aTargetSize)
 {
-    SdrMark* pM=GetMarkedObjectList().GetMark(0);
-    SdrObject* pO=pM->GetMarkedSdrObj();
-    if (!pO->IsSizeValid(aTargetSize))
-        return false;
-    return true;
+    if (SdrMark* pM=GetMarkedObjectList().GetMark(0))
+    {
+        if (SdrObject* pO=pM->GetMarkedSdrObj())
+        {
+            if (pO->IsSizeValid(aTargetSize))
+                return true;
+        }
+        else
+            SAL_WARN("svx", "no MarkedSdrObj found");
+    }
+    else
+        SAL_WARN("svx", "no SdrMark found");
+
+    return false;
 }
 
 void SdrEditView::ResizeMarkedObj(const Point& rRef, const Fraction& xFact, 
const Fraction& yFact, bool bCopy)

Reply via email to