sd/source/ui/remotecontrol/Communicator.cxx | 1 - 1 file changed, 1 deletion(-)
New commits: commit e5aa82b259e35e3976734af38fff643ea28767df Author: dldld <[email protected]> AuthorDate: Mon Oct 17 17:52:01 2022 +0200 Commit: Mike Kaganski <[email protected]> CommitDate: Tue Oct 25 13:32:48 2022 +0200 tdf#151608: Fix crash in disposing remotecontrol listener Clearing the listener twice seems to be the problem here, because the WeakComponentImplHelperBase::dispose() tries to access the cleared listener when it was already cleared, causing the crash. The problem which is fixed by this commit appears when the Impress Remote App is closed, which is connected to a running presentation. So closing the app "kills" the presentation. I think I have seen this also happening, when the connection was closed due to bad wifi. Which also makes sense as the communication thread is getting stopped and the listener will be destroyed. Change-Id: I3c8fa535cfd4d82c846494f76ffb9ce43ebd606a Reviewed-on: https://gerrit.libreoffice.org/c/core/+/141468 Reviewed-by: Mike Kaganski <[email protected]> Tested-by: Mike Kaganski <[email protected]> diff --git a/sd/source/ui/remotecontrol/Communicator.cxx b/sd/source/ui/remotecontrol/Communicator.cxx index b0645e72d430..59509ed3c74b 100644 --- a/sd/source/ui/remotecontrol/Communicator.cxx +++ b/sd/source/ui/remotecontrol/Communicator.cxx @@ -130,7 +130,6 @@ void Communicator::informListenerDestroyed() if ( pTransmitter ) pTransmitter->addMessage( "slideshow_finished\n\n", Transmitter::PRIORITY_HIGH ); - mListener.clear(); } void Communicator::presentationStarted( const css::uno::Reference<
