diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2020-01-28 10:39:16 +0100 |
---|---|---|
committer | Thorsten Behrens <thorsten.behrens@allotropia.de> | 2021-04-09 23:22:06 +0200 |
commit | 56df72bcc6367945bc379ebabb226035c78482db (patch) | |
tree | 5d383560a7323457ed4ba98322969b2194695200 /desktop | |
parent | vcl: removed unit test added during pdfium update (diff) | |
download | core-56df72bcc6367945bc379ebabb226035c78482db.tar.gz core-56df72bcc6367945bc379ebabb226035c78482db.zip |
Dispose configmgr at the end of unopkg
...so that the configmgr::Components::WriteThread
(configmgr/source/components.cxx) is joined well before exit. (This should
solve the issue discussed at <https://gerrit.libreoffice.org/c/core/+/86543/3/
configmgr/source/writemodfile.cxx#564>, "tdf#129917 Use temp user profile when
installing shared extensions".)
Change-Id: I79004c2e7f839d80eda23eed2a1c5e8e660a58df
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/87592
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
(cherry picked from commit 9f277b2616512299ec20de0209c272527f26386b)
Diffstat (limited to 'desktop')
-rw-r--r-- | desktop/source/pkgchk/unopkg/unopkg_app.cxx | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/desktop/source/pkgchk/unopkg/unopkg_app.cxx b/desktop/source/pkgchk/unopkg/unopkg_app.cxx index f6e4d0c19b7c..35d2bee6f5c5 100644 --- a/desktop/source/pkgchk/unopkg/unopkg_app.cxx +++ b/desktop/source/pkgchk/unopkg/unopkg_app.cxx @@ -590,6 +590,8 @@ extern "C" int unopkg_main() logger->log(LogLevel::INFO, "$1$ done.", APP_NAME); //Force to release all bridges which connect us to the child processes dp_misc::disposeBridges(xLocalComponentContext); + css::uno::Reference<css::lang::XComponent>( + xLocalComponentContext, css::uno::UNO_QUERY_THROW)->dispose(); return 0; } catch (const ucb::CommandFailedException &e) @@ -621,6 +623,10 @@ extern "C" int unopkg_main() if (!bNoOtherErrorMsg) logger->log(LogLevel::SEVERE, "$1$ failed.", APP_NAME); dp_misc::disposeBridges(xLocalComponentContext); + if (xLocalComponentContext.is()) { + css::uno::Reference<css::lang::XComponent>( + xLocalComponentContext, css::uno::UNO_QUERY_THROW)->dispose(); + } return 1; } |