diff options
| author | Morten Johan Sørvig <morten.sorvig@theqtcompany.com> | 2015-08-18 11:39:01 +0200 |
|---|---|---|
| committer | Morten Johan Sørvig <morten.sorvig@theqtcompany.com> | 2015-09-02 07:28:18 +0000 |
| commit | 111ee58c065602bfcde520b54f44809d854e73f0 (patch) | |
| tree | fdf5c0875247708e964d67f6fcbc0f70084cbe68 /src/widgets/kernel/qgesturemanager_p.h | |
| parent | 6f9e845d9b02ed2becaab4af837535173dad2068 (diff) | |
Prevent QGestureManager crash.
Use QPointer to prevent dereferencing stale pointers
stored in m_gestureTargets.
Add null pointer tests in addition to the Q_ASSERTs
on the pointers returned by m_gestureTargets.value().
The intention is to assert in debug mode but keep
going in release mode.
Change-Id: Icdef8cc02040bddc88f4bcb268e9ca0ac252557d
Task-number: QTBUG-46264
Reviewed-by: Pawel Kurdybacha <pawel.kurdybacha@gmail.com>
Reviewed-by: Gabriel de Dietrich <gabriel.dedietrich@theqtcompany.com>
Diffstat (limited to 'src/widgets/kernel/qgesturemanager_p.h')
| -rw-r--r-- | src/widgets/kernel/qgesturemanager_p.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/widgets/kernel/qgesturemanager_p.h b/src/widgets/kernel/qgesturemanager_p.h index 8ba253d17e..4e349ac731 100644 --- a/src/widgets/kernel/qgesturemanager_p.h +++ b/src/widgets/kernel/qgesturemanager_p.h @@ -117,7 +117,7 @@ private: QHash<QGesture *, QGestureRecognizer *> m_gestureToRecognizer; QHash<QGesture *, QObject *> m_gestureOwners; - QHash<QGesture *, QWidget *> m_gestureTargets; + QHash<QGesture *, QPointer<QWidget> > m_gestureTargets; int m_lastCustomGestureId; |