diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2015-07-02 16:40:21 +0100 |
|---|---|---|
| committer | Miklos Vajna <vmiklos@collabora.co.uk> | 2015-07-03 10:35:33 +0000 |
| commit | 338f47cf52f6eed3c05021d4272604438f0f550d (patch) | |
| tree | ba14d0d734b168e366c880c15d097e31db5ee87b /sc | |
| parent | 6f4c31eb757cd19311b23ce06fdbbe9268f9cd16 (diff) | |
fix a11y crash seen on moving chart wizard dialog
Change-Id: Ic3ba292e28fe12d7dcc2c2e67aeea48a4c8aaac2
(cherry picked from commit b161552bd9f7d6b6de9752e4f0e7d6f65bbcf42e)
(cherry picked from commit ea228fdffd17b87e398216625213a691fcb34825)
Reviewed-on: https://gerrit.libreoffice.org/16714
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Tested-by: Miklos Vajna <vmiklos@collabora.co.uk>
Diffstat (limited to 'sc')
| -rw-r--r-- | sc/source/ui/Accessibility/AccessibleDocument.cxx | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/sc/source/ui/Accessibility/AccessibleDocument.cxx b/sc/source/ui/Accessibility/AccessibleDocument.cxx index ea2688eb196d..b2ff3253cde8 100644 --- a/sc/source/ui/Accessibility/AccessibleDocument.cxx +++ b/sc/source/ui/Accessibility/AccessibleDocument.cxx @@ -813,8 +813,9 @@ uno::Reference< XAccessible > ScChildrenShapes::GetSelected(sal_Int32 nSelectedC std::vector < uno::Reference < drawing::XShape > > aShapes; FillShapes(aShapes); - if(aShapes.size()<=0) + if (nSelectedChildIndex < 0 || static_cast<size_t>(nSelectedChildIndex) >= aShapes.size()) return xAccessible; + SortedShapes::iterator aItr; if (FindShape(aShapes[nSelectedChildIndex], aItr)) xAccessible = Get(aItr - maZOrderedShapes.begin()); |