diff options
author | akash <akash96j@gmail.com> | 2016-03-25 10:09:07 +0530 |
---|---|---|
committer | Michael Stahl <mstahl@redhat.com> | 2016-03-30 20:45:20 +0000 |
commit | f80de31099be1ba7c6f753187f6703a915151c27 (patch) | |
tree | 6843807300943ddebf43beaa4b5121b625bd3ac5 | |
parent | loplugin:nullptr: Find some more cases in templates (diff) | |
download | core-f80de31099be1ba7c6f753187f6703a915151c27.tar.gz core-f80de31099be1ba7c6f753187f6703a915151c27.zip |
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>
-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 178a92eba3b3..9441e5c66383 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 () |