[Libreoffice-commits] core.git: sc/source
Libreoffice Gerrit user
logerrit at kemper.freedesktop.org
Thu Oct 11 20:57:20 UTC 2018
sc/source/ui/miscdlgs/datatableview.cxx | 2 ++
1 file changed, 2 insertions(+)
New commits:
commit e6af1ef9836199724987fbe86dc49dd4401c5dfa
Author: Mike Kaganski <mike.kaganski at collabora.com>
AuthorDate: Thu Oct 11 22:40:56 2018 +0300
Commit: Mike Kaganski <mike.kaganski at collabora.com>
CommitDate: Thu Oct 11 22:56:56 2018 +0200
tdf#120528: check for empty mpMouseEvent in ScDataTableView::MouseButtonUp
The preceeding mouse button down event could happen in a different (closing)
window, like context menu.
Change-Id: I630b827fb5fe05399ca8436ea79210f4642a56d4
Reviewed-on: https://gerrit.libreoffice.org/61691
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski at collabora.com>
diff --git a/sc/source/ui/miscdlgs/datatableview.cxx b/sc/source/ui/miscdlgs/datatableview.cxx
index 174342653d93..897e7a31cdb4 100644
--- a/sc/source/ui/miscdlgs/datatableview.cxx
+++ b/sc/source/ui/miscdlgs/datatableview.cxx
@@ -243,6 +243,8 @@ void ScDataTableView::MouseButtonUp(const MouseEvent& rMEvt)
{
if (!rMEvt.IsLeft())
return;
+ if (!mpMouseEvent) // tdf#120528 The event originated in another window, like context menu
+ return;
SCCOL nStartCol = findColFromPos(mpMouseEvent->GetPosPixel().getX(), mpDoc.get());
SCCOL nEndCol = findColFromPos(rMEvt.GetPosPixel().getX(), mpDoc.get());
More information about the Libreoffice-commits
mailing list