diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2015-08-13 16:58:58 +0200 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2015-08-13 17:08:22 +0200 |
| commit | 07ebec47da4a096aaab53bf6a90bb219ce29bfbf (patch) | |
| tree | 55b125e7256c576e19415e319b84ea342aeb7c6b /vcl/unx/gtk/a11y/atkregistry.hxx | |
| parent | dd8fbd4c8ae18421975f45c7aa7befbf74f0b0d7 (diff) | |
rhbz#1247588: Hold UNO objects by css::uno::Reference
...instead of raw pointer and manual acquire/relase. It is unclear to me why
the original code thought it was necessary (or merely "better") to hold by raw
pointer; but at least from the backtrace in rhbz#1247588, it seems plausible
that UNO method calls through such raw pointers could recursively call into
atk_object_wrapper_dispose and make the raw pointer stale.
Change-Id: Idc0a4f9e2f7ffe610261c1b7b98ce9c5e040db43
Diffstat (limited to 'vcl/unx/gtk/a11y/atkregistry.hxx')
| -rw-r--r-- | vcl/unx/gtk/a11y/atkregistry.hxx | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/vcl/unx/gtk/a11y/atkregistry.hxx b/vcl/unx/gtk/a11y/atkregistry.hxx index c5f70a8528e9..b25e449df474 100644 --- a/vcl/unx/gtk/a11y/atkregistry.hxx +++ b/vcl/unx/gtk/a11y/atkregistry.hxx @@ -27,7 +27,8 @@ AtkObject * ooo_wrapper_registry_get(const ::com::sun::star::uno::Reference< ::c void ooo_wrapper_registry_add(const ::com::sun::star::uno::Reference< ::com::sun::star::accessibility::XAccessible >& rxAccessible, AtkObject *obj); -void ooo_wrapper_registry_remove(::com::sun::star::accessibility::XAccessible *pAccessible); +void ooo_wrapper_registry_remove( + css::uno::Reference<css::accessibility::XAccessible> const & pAccessible); #endif // __ATK_REGISTRY_HXX_ |