diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2021-12-24 20:15:53 +0200 |
---|---|---|
committer | Xisco Fauli <xiscofauli@libreoffice.org> | 2022-01-11 10:29:38 +0100 |
commit | 02e7628f603959d15c0f1b8a8d2c17b1550479b3 (patch) | |
tree | bad340e0520d27ed5e0789fc0999754d29930b33 | |
parent | b1e3f5d96b1093f42a4558a04ee607b2cde6f4bd (diff) |
tdf#145090 LO Calc Crash with drop-down-menu
avoid attempting to call into widgets while in the middle of dispose
regression from
commit 8d485ec0cd35ee1ae7684f2b6ca96c0f0c6f9dac
IsDisposed->isDisposed in vcl/../window
Change-Id: If346dc9dc8547f8da28d091bfa20eff39162a0b1
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127437
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
(cherry picked from commit 3454055d4da3326540905fc69626b934d5aa0744)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127483
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
(cherry picked from commit 0c183add008e3604711d1995716cdcffc0881dc6)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127577
Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>
-rw-r--r-- | comphelper/source/misc/accessiblewrapper.cxx | 4 | ||||
-rw-r--r-- | include/comphelper/accessiblewrapper.hxx | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/comphelper/source/misc/accessiblewrapper.cxx b/comphelper/source/misc/accessiblewrapper.cxx index 9aa1b48e2f1e..1167f56403c3 100644 --- a/comphelper/source/misc/accessiblewrapper.cxx +++ b/comphelper/source/misc/accessiblewrapper.cxx @@ -86,7 +86,7 @@ namespace comphelper Reference< XAccessible > OWrappedAccessibleChildrenManager::getAccessibleWrapperFor( const Reference< XAccessible >& _rxKey ) { - Reference< XAccessible > xValue; + rtl::Reference< OAccessibleWrapper > xValue; if( !_rxKey.is() ) { @@ -140,7 +140,7 @@ namespace comphelper Reference< XComponent > xContextComponent; if( rChild.second.is() ) - xContextComponent.set( rChild.second->getAccessibleContext(), + xContextComponent.set( rChild.second->getContextNoCreate(), ::css::uno::UNO_QUERY ); if( xContextComponent.is() ) xContextComponent->dispose(); diff --git a/include/comphelper/accessiblewrapper.hxx b/include/comphelper/accessiblewrapper.hxx index ea9a0af0209b..f73c312c3ab5 100644 --- a/include/comphelper/accessiblewrapper.hxx +++ b/include/comphelper/accessiblewrapper.hxx @@ -316,7 +316,7 @@ namespace comphelper typedef ::std::map < css::uno::Reference< css::accessibility::XAccessible > - , css::uno::Reference< css::accessibility::XAccessible > + , rtl::Reference< comphelper::OAccessibleWrapper > > AccessibleMap; // TODO: think about if we should hold these objects weak |