diff options
| author | akash <akash96j@gmail.com> | 2016-04-01 23:36:35 +0530 |
|---|---|---|
| committer | Michael Stahl <mstahl@redhat.com> | 2016-04-05 14:49:35 +0000 |
| commit | 1087ac3755da09f20a04f451a781f4fc8b53a680 (patch) | |
| tree | 00297975dc4fa08526c78a12ad69502221e5c629 | |
| parent | 062d0b1e36217a88ea3a727a14df68bd81f6092e (diff) | |
Prevent broadcast of title change events without a source
Do not send title changed event if the source of the event is null.
This case can occur in multi-threaded scenario where one thread calls
disposing while some other thread calls impl_sendTitleChangedEvent().
Change-Id: I86a65824ab5c60a3fd8d3fb323f6d78944b57a23
Reviewed-on: https://gerrit.libreoffice.org/23736
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>
| -rw-r--r-- | framework/source/fwe/helper/titlehelper.cxx | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/framework/source/fwe/helper/titlehelper.cxx b/framework/source/fwe/helper/titlehelper.cxx index 52cb4198b8ad..cbd18f96af17 100644 --- a/framework/source/fwe/helper/titlehelper.cxx +++ b/framework/source/fwe/helper/titlehelper.cxx @@ -268,6 +268,9 @@ void TitleHelper::impl_sendTitleChangedEvent () aLock.clear (); // <- SYNCHRONIZED + if( ! (aEvent.Source).is() ) + return; + ::cppu::OInterfaceContainerHelper* pContainer = m_aListener.getContainer( cppu::UnoType<css::frame::XTitleChangeListener>::get()); if ( ! pContainer) return; |