diff options
| author | Morten Johan Sørvig <morten.sorvig@qt.io> | 2020-12-09 16:31:41 +0100 |
|---|---|---|
| committer | Morten Johan Sørvig <morten.sorvig@qt.io> | 2020-12-21 22:48:29 +0000 |
| commit | b3b95670b2f7e43cfe49fa90f2dffa74b82c160f (patch) | |
| tree | c6528b933db54bd942ceb95a54ea44d182f433b9 /src/corelib/thread/qthread.cpp | |
| parent | cd6d805d9368a0e84cc20a416ab88603c4be6a9c (diff) | |
Call QEventDispatcher::startingUp() on thread start
...instead of during createEventDispatcher().
This way, startingUp() will be called [on the thread
being started] also for custom event dispatchers installed
with QThread::setEventDispatcher().
This prevents crashes when installing event dispatches which
expects that startingUp() will be called.
Crash reproducible with test case from QTBUG-51961.
Change-Id: I71bd696539689d928a61ff9b47d05297cf803b2f
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
Diffstat (limited to 'src/corelib/thread/qthread.cpp')
| -rw-r--r-- | src/corelib/thread/qthread.cpp | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/corelib/thread/qthread.cpp b/src/corelib/thread/qthread.cpp index 543c491201..3453fd0fc1 100644 --- a/src/corelib/thread/qthread.cpp +++ b/src/corelib/thread/qthread.cpp @@ -125,7 +125,6 @@ QAbstractEventDispatcher *QThreadData::createEventDispatcher() { QAbstractEventDispatcher *ed = QThreadPrivate::createEventDispatcher(this); eventDispatcher.storeRelease(ed); - ed->startingUp(); return ed; } |