[Libreoffice-commits] core.git: sc/source

[Libreoffice Gerrit user]

 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());