diff options
author | akash <akash96j@gmail.com> | 2016-03-25 10:09:07 +0530 |
---|---|---|
committer | Miklos Vajna <vmiklos@collabora.co.uk> | 2016-04-19 08:11:28 +0000 |
commit | 0dbd5b7195669225bbe2aafb53aeed577394170e (patch) | |
tree | 5432b7578bac80c8756cb7d15269e410be9759ab | |
parent | 2407399c0c840c2e3d9de6575a99695967e88039 (diff) |
tdf#96896-Fix infinite recursion to prevent Writer crash
To prevent infinite recursion set m_xOwner to null.
Removed impl_sendTitleChangedEvent from disposing method ensuring that
title is never constructed for a component that is disposing.
Change-Id: I9bc20a9a582d5d911509d2aa30fb0d0122eb588e
Reviewed-on: https://gerrit.libreoffice.org/23509
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>
(cherry picked from commit f80de31099be1ba7c6f753187f6703a915151c27)
Reviewed-on: https://gerrit.libreoffice.org/24138
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
-rw-r--r-- | framework/source/fwe/helper/titlehelper.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/framework/source/fwe/helper/titlehelper.cxx b/framework/source/fwe/helper/titlehelper.cxx index c9aba3e5b2e7..c7b5bf482f86 100644 --- a/framework/source/fwe/helper/titlehelper.cxx +++ b/framework/source/fwe/helper/titlehelper.cxx @@ -249,13 +249,13 @@ void SAL_CALL TitleHelper::disposing(const css::lang::EventObject& aEvent) // SYNCHRONIZED -> aLock.reset (); + m_xOwner = nullptr; m_sTitle = OUString (); m_nLeasedNumber = css::frame::UntitledNumbersConst::INVALID_NUMBER; aLock.clear (); // <- SYNCHRONIZED - impl_sendTitleChangedEvent (); } void TitleHelper::impl_sendTitleChangedEvent () |