[Libreoffice-commits] core.git: sw/source

Jim Raykowski (via logerrit) logerrit at kemper.freedesktop.org
Fri Jul 5 09:38:51 UTC 2019


 sw/source/uibase/docvw/HeaderFooterWin.cxx |    5 ++---
 sw/source/uibase/wrtsh/wrtsh1.cxx          |    2 ++
 2 files changed, 4 insertions(+), 3 deletions(-)

New commits:
commit 151eae314358f9ee0df017cb89fbc8c87c969cdb
Author:     Jim Raykowski <raykowj at gmail.com>
AuthorDate: Tue Jul 2 08:54:02 2019 -0800
Commit:     Michael Stahl <Michael.Stahl at cib.de>
CommitDate: Fri Jul 5 11:37:58 2019 +0200

    tdf#125863 Rework tdf#107593 so header footer delete doesn't crash
    
    Header footer delete from the last page when last page is unneeded
    (stack level 24 SwRootFrame::RemoveSuperfluous /// remove pages that are
    not needed at all) causes SwFrameMenuButtonBase::dispose() which sets
    m_pFrame to nullptr which causes crash when IsEmptyHeaderFooter calls
    GetPageFrame(). Doing ToggleHeaderFooterEdit() when header or footer
    deletion is accepted in SwWrtShell::ChangeHeaderOrFooter avoids the
    crash. SwHeaderFooterWin is also disposed which would make the
    GrabFocusToDocument crash if it got that far so use the shell window
    which is still valid here to GrabFocusToDocument.
    
    Change-Id: Iaced23e6716442671c5ea29b79be178f4fc836d3
    Reviewed-on: https://gerrit.libreoffice.org/75010
    Reviewed-by: Michael Stahl <Michael.Stahl at cib.de>
    Tested-by: Michael Stahl <Michael.Stahl at cib.de>

diff --git a/sw/source/uibase/docvw/HeaderFooterWin.cxx b/sw/source/uibase/docvw/HeaderFooterWin.cxx
index e22530d780ea..45d84a630281 100644
--- a/sw/source/uibase/docvw/HeaderFooterWin.cxx
+++ b/sw/source/uibase/docvw/HeaderFooterWin.cxx
@@ -448,9 +448,8 @@ void SwHeaderFooterWin::ExecuteCommand(const OString& rIdent)
     else if (rIdent == "delete")
     {
         rSh.ChangeHeaderOrFooter( rStyleName, m_bIsHeader, false, true );
-        if ( IsEmptyHeaderFooter() )
-            rSh.ToggleHeaderFooterEdit();
-        GrabFocusToDocument();
+        // warning: "this" may be disposed now
+        rSh.GetWin()->GrabFocusToDocument();
     }
     else if (rIdent == "insert_pagenumber")
     {
diff --git a/sw/source/uibase/wrtsh/wrtsh1.cxx b/sw/source/uibase/wrtsh/wrtsh1.cxx
index ec000c050552..fa1db7f69e70 100644
--- a/sw/source/uibase/wrtsh/wrtsh1.cxx
+++ b/sw/source/uibase/wrtsh/wrtsh1.cxx
@@ -1825,6 +1825,8 @@ void SwWrtShell::ChangeHeaderOrFooter(
 
                 bExecute = nResult == RET_YES;
                 StartAllAction();
+                if (nResult == RET_YES)
+                    ToggleHeaderFooterEdit();
             }
             if( bExecute )
             {


More information about the Libreoffice-commits mailing list