sd/source/ui/inc/SlideSorterViewShell.hxx |    2 +-
 sd/source/ui/view/drviews2.cxx            |    7 ++++++-
 sd/source/ui/view/drviewse.cxx            |    7 +++----
 3 files changed, 10 insertions(+), 6 deletions(-)

New commits:
commit 48b650d50070cd1bff391aef53f781c16899d92d
Author:     Xisco Fauli <xiscofa...@libreoffice.org>
AuthorDate: Thu Sep 11 17:17:34 2025 +0200
Commit:     Christian Lohmaier <lohmaier+libreoff...@googlemail.com>
CommitDate: Wed Sep 17 18:58:06 2025 +0200

    sd: use SAL_RET_MAYBENULL in GetSlideSorter
    
    Change-Id: I2f71942aeaa7aa1cc78be69d63a94588099ad4d8
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/190821
    Tested-by: Jenkins
    Reviewed-by: Xisco Fauli <xiscofa...@libreoffice.org>
    Signed-off-by: Xisco Fauli <xiscofa...@libreoffice.org>
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/190877
    Reviewed-by: Christian Lohmaier <lohmaier+libreoff...@googlemail.com>

diff --git a/sd/source/ui/inc/SlideSorterViewShell.hxx 
b/sd/source/ui/inc/SlideSorterViewShell.hxx
index cb2533740b66..ef446ff12f06 100644
--- a/sd/source/ui/inc/SlideSorterViewShell.hxx
+++ b/sd/source/ui/inc/SlideSorterViewShell.hxx
@@ -65,7 +65,7 @@ public:
         returned.
     */
     // Exported for unit test
-    SD_DLLPUBLIC static SlideSorterViewShell* GetSlideSorter(ViewShellBase& 
rBase);
+    SD_DLLPUBLIC SAL_RET_MAYBENULL static SlideSorterViewShell* 
GetSlideSorter(ViewShellBase& rBase);
 
     virtual SdPage* GetActualPage() override;
 
diff --git a/sd/source/ui/view/drviews2.cxx b/sd/source/ui/view/drviews2.cxx
index c709ffdb4733..6f9fa6fba559 100644
--- a/sd/source/ui/view/drviews2.cxx
+++ b/sd/source/ui/view/drviews2.cxx
@@ -4558,11 +4558,16 @@ SdPage* DrawViewShell::CreateOrDuplicatePage (
 
 void DrawViewShell::DuplicateSelectedSlides (SfxRequest& rRequest)
 {
+    ::sd::slidesorter::SlideSorterViewShell* pSlideSorterViewShell =
+        
sd::slidesorter::SlideSorterViewShell::GetSlideSorter(GetViewShellBase());
+    if (!pSlideSorterViewShell)
+        return;
+
     // Create a list of the pages that are to be duplicated.  The process of
     // duplication alters the selection.
     sal_Int32 nInsertPosition (0);
     ::std::vector<SdPage*> aPagesToDuplicate;
-    sd::slidesorter::SlideSorter &mrSlideSorter = 
sd::slidesorter::SlideSorterViewShell::GetSlideSorter(GetViewShellBase())->GetSlideSorter();
+    sd::slidesorter::SlideSorter &mrSlideSorter = 
pSlideSorterViewShell->GetSlideSorter();
     sd::slidesorter::model::PageEnumeration aSelectedPages (
         
sd::slidesorter::model::PageEnumerationProvider::CreateSelectedPagesEnumeration(mrSlideSorter.GetModel()));
     while (aSelectedPages.HasMoreElements())
diff --git a/sd/source/ui/view/drviewse.cxx b/sd/source/ui/view/drviewse.cxx
index 55fcc2c1ba3b..1b77e8a51580 100644
--- a/sd/source/ui/view/drviewse.cxx
+++ b/sd/source/ui/view/drviewse.cxx
@@ -957,10 +957,9 @@ void DrawViewShell::FuSupport(SfxRequest& rReq)
         case SID_PASTE_SLIDE:
         case SID_COPY_SLIDE:
         {
-            
sd::slidesorter::SlideSorterViewShell::GetSlideSorter(GetViewShellBase())
-                ->GetSlideSorter()
-                .GetController()
-                .FuSupport(rReq);
+            if (::sd::slidesorter::SlideSorterViewShell* pSlideSorterViewShell 
=
+                
sd::slidesorter::SlideSorterViewShell::GetSlideSorter(GetViewShellBase()))
+                    
pSlideSorterViewShell->GetSlideSorter().GetController().FuSupport(rReq);
             Cancel();
             rReq.Done();
         }

Reply via email to