summaryrefslogtreecommitdiff
path: root/vcl/source/window/winproc.cxx
diff options
context:
space:
mode:
authorCaolán McNamara <caolanm@redhat.com>2016-01-26 14:46:49 +0000
committerCaolán McNamara <caolanm@redhat.com>2016-01-26 14:47:09 +0000
commit5d29ed1801a07d4649e095c25935b50f5ad32eb4 (patch)
treebd0288d3c29f34505d7a9f6c1b08c09a713a51f2 /vcl/source/window/winproc.cxx
parent4cd058f3f6568724c65ac88769f17fcc6078da23 (diff)
Resolves: tdf#97331 use VclPtr instead of pointers to avoid crash
Change-Id: Ia653a67046cb2cfb7c96367a7483ddc0cb29819e
Diffstat (limited to 'vcl/source/window/winproc.cxx')
-rw-r--r--vcl/source/window/winproc.cxx14
1 files changed, 7 insertions, 7 deletions
diff --git a/vcl/source/window/winproc.cxx b/vcl/source/window/winproc.cxx
index 9e9a7fcd050b..e484f7d46511 100644
--- a/vcl/source/window/winproc.cxx
+++ b/vcl/source/window/winproc.cxx
@@ -1314,7 +1314,7 @@ static bool ImplCallWheelCommand( const VclPtr<vcl::Window>& pWindow, const Poin
static bool acceptableWheelScrollTarget(const vcl::Window *pMouseWindow)
{
- return (pMouseWindow && pMouseWindow->IsInputEnabled() && !pMouseWindow->IsInModalMode());
+ return (pMouseWindow && !pMouseWindow->isDisposed() && pMouseWindow->IsInputEnabled() && !pMouseWindow->IsInModalMode());
}
//If the last event at the same absolute screen position was handled by a
@@ -1480,26 +1480,26 @@ public:
bool HandleWheelEvent::HandleEvent(const SalWheelMouseEvent& rEvt)
{
static SalWheelMouseEvent aPreviousEvent;
- static vcl::Window *pPreviousWindow;
+ static VclPtr<vcl::Window> xPreviousWindow;
if (!Setup())
return false;
- vcl::Window *pMouseWindow = FindTarget();
+ VclPtr<vcl::Window> xMouseWindow = FindTarget();
// avoid the problem that scrolling via wheel to this point brings a widget
// under the mouse that also accepts wheel commands, so stick with the old
// widget if the time gap is very small
- if (shouldReusePreviousMouseWindow(aPreviousEvent, rEvt) && acceptableWheelScrollTarget(pPreviousWindow))
+ if (shouldReusePreviousMouseWindow(aPreviousEvent, rEvt) && acceptableWheelScrollTarget(xPreviousWindow))
{
- pMouseWindow = pPreviousWindow;
+ xMouseWindow = xPreviousWindow.get();
}
aPreviousEvent = rEvt;
- pPreviousWindow = Dispatch(pMouseWindow);
+ xPreviousWindow = Dispatch(xMouseWindow);
- return pPreviousWindow != nullptr;
+ return xPreviousWindow;
}
class HandleGestureEvent : public HandleGestureEventBase