commit 62a50239e06ac2f94f4a1cfbef5a8689bd42b875
author Stephan Bergmann <stephan.bergmann@allotropia.de> Fri Dec 15 14:40:53 2023 +0100
committer Stephan Bergmann <stephan.bergmann@allotropia.de> Sat Dec 16 10:54:01 2023 +0100
tree 88d526652bf4020641fdd298d63c6d00ac1b44bb
parent b03aa9647c860af7948f219ed9a0e90e5eb374d5

Avoid "Desktop disposed before terminating it" assert

...from Desktop::disposing() at framework/source/services/desktop.cxx:1041, on
Windows, when the HAVE_FEATURE_UPDATE_MAR update() call returns true (and
soffice.bin is about to exit now), but the css.frame.Desktop service happens to
already have been created somewhere else (and will thus be disposed as part of
disposing the service manager) but doesn't receive a terminate() call in
Desktop::Main().

So reorder the code in Desktop::Main(), moving the HAVE_FEATURE_UPDATE_MAR code
after the

>     xDesktop = css::frame::Desktop::create( xContext );

so that we have something to call

>                 xDesktop->terminate();

on in this scenario.

Change-Id: Iee5b6eeece33d5a27cadb94a7da0d26aa1492e91
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/160831
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <stephan.bergmann@allotropia.de>
(cherry picked from commit 98caf4786033da6c39b4f746d79ab4f340683d32)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/160850

desktop/source/app/app.cxx

diff --git a/desktop/source/app/app.cxx b/desktop/source/app/app.cxx
index 4c13a2a..8136d39 100644
--- a/desktop/source/app/app.cxx
+++ b/desktop/source/app/app.cxx
@@ -1351,6 +1351,22 @@ int Desktop::Main()
    if ( !InitializeConfiguration() )
        return EXIT_FAILURE;

    SetSplashScreenProgress(30);

    // create title string
    OUString aTitle(ReplaceStringHookProc(RID_APPTITLE));
#ifdef DBG_UTIL
    //include buildid in non product builds
    aTitle += " [" + utl::Bootstrap::getBuildIdData("development") + "]";
#endif

    SetDisplayName( aTitle );
    SetSplashScreenProgress(35);
    pExecGlobals->pPathOptions.reset( new SvtPathOptions);
    SetSplashScreenProgress(40);

    xDesktop = css::frame::Desktop::create( xContext );

#if HAVE_FEATURE_UPDATE_MAR
    const char* pUpdaterTestEnable = std::getenv("LIBO_UPDATER_TEST_ENABLE");
    if (pUpdaterTestEnable || officecfg::Office::Update::Update::Enabled::get())
@@ -1414,7 +1430,10 @@ int Desktop::Main()
            CloseSplashScreen();
            bool bSuccess = update();
            if (bSuccess)
            {
                xDesktop->terminate();
                return EXIT_SUCCESS;
            }
        }
        else if (isTimeForUpdateCheck() || pForcedUpdateCheck)
        {
@@ -1429,22 +1448,6 @@ int Desktop::Main()
    }
#endif

    SetSplashScreenProgress(30);

    // create title string
    OUString aTitle(ReplaceStringHookProc(RID_APPTITLE));
#ifdef DBG_UTIL
    //include buildid in non product builds
    aTitle += " [" + utl::Bootstrap::getBuildIdData("development") + "]";
#endif

    SetDisplayName( aTitle );
    SetSplashScreenProgress(35);
    pExecGlobals->pPathOptions.reset( new SvtPathOptions);
    SetSplashScreenProgress(40);

    xDesktop = css::frame::Desktop::create( xContext );

    // create service for loading SFX (still needed in startup)
    pExecGlobals->xGlobalBroadcaster = Reference < css::document::XDocumentEventListener >
        ( css::frame::theGlobalEventBroadcaster::get(xContext), UNO_SET_THROW );