Kaydet (Commit) 459a611c authored tarafından Eike Rathke's avatar Eike Rathke

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 0a18c1dc
CommitDate: Fri Dec 21 18:52:31 2018 +0100

    pass ScAddress around using boost::optional

Change-Id: Id676b2d84d324b989259688069d4860de5559ebc
Reviewed-on: https://gerrit.libreoffice.org/72605Reviewed-by: 's avatarEike Rathke <erack@redhat.com>
Tested-by: Jenkins
Reviewed-by: 's avatarNoel Grandin <noel.grandin@collabora.co.uk>
üst e8d5b8be
......@@ -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));
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment