diff options
author | Eike Rathke <erack@redhat.com> | 2019-05-20 16:25:26 +0200 |
---|---|---|
committer | Eike Rathke <erack@redhat.com> | 2019-05-20 20:20:26 +0200 |
commit | 459a611c9933c3b94c421b3f6036cd2e926279aa (patch) | |
tree | dcf0729a71f3723dab10c38cff23d02bb8bd6e5c /sc/source/ui/Accessibility | |
parent | e8d5b8beb5958147235ff955ed38c47b51d860ff (diff) |
Fix a11y crash with drag&drop shape anchored to page
Dereferencing an empty boost::optional is dereferencing a this
nullptr, simply copy it instead.
Regression from
commit 0a18c1dc437642dd7ec3631367a9aab9982c55de
CommitDate: Fri Dec 21 18:52:31 2018 +0100
pass ScAddress around using boost::optional
Change-Id: Id676b2d84d324b989259688069d4860de5559ebc
Reviewed-on: https://gerrit.libreoffice.org/72605
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sc/source/ui/Accessibility')
-rw-r--r-- | sc/source/ui/Accessibility/AccessibleDocument.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sc/source/ui/Accessibility/AccessibleDocument.cxx b/sc/source/ui/Accessibility/AccessibleDocument.cxx index 9ad7aa3a98f0..565de0e65fec 100644 --- a/sc/source/ui/Accessibility/AccessibleDocument.cxx +++ b/sc/source/ui/Accessibility/AccessibleDocument.cxx @@ -1140,7 +1140,7 @@ void ScChildrenShapes::SetAnchor(const uno::Reference<drawing::XShape>& xShape, if ((xAddress && pData->xRelationCell && (*xAddress != *(pData->xRelationCell))) || (!xAddress && pData->xRelationCell) || (xAddress && !pData->xRelationCell)) { - pData->xRelationCell = *xAddress; + pData->xRelationCell = xAddress; if (pData->pAccShape.is()) pData->pAccShape->SetRelationSet(GetRelationSet(pData)); } |