diff options
author | Marc Mutz <marc.mutz@kdab.com> | 2019-07-04 14:08:44 +0200 |
---|---|---|
committer | Marc Mutz <marc.mutz@kdab.com> | 2019-07-05 18:50:12 +0000 |
commit | 90a29a73f84ffd6386beb37690c328b039423fab (patch) | |
tree | eb7af8f930cfe073dad8ad7f82079d2f75ce53a1 /src/network/kernel/qdnslookup.cpp | |
parent | 85dc392135feb72eed449e6eaf47cdd023879394 (diff) |
QHostInfo: fix a race condition on wasDeleted
The plain bool variable wasDeleted is set to true in the
QHostLookupManager dtor before the call to clear(), which calls
waitForDone() on the thread pool performing the lookups. All tasks on
the thread pool start by checking this variable so as to return early
when destruction is in progress. But the check was outside the
mutex-protected area, so as a non-atomic load, without a
happens-before relation to the write, this is a Data Race, thus UB.
Fix by moving the check past the mutex locking into the critical
section. This way, tasks that were waiting for the mutex after seeing
no wasDeleted before get the message reliably.
This does not introduce a dead-lock, since the call to waitForDone()
is outside any mutex protection leaving a window for the tasks to
obtain the mutex and react on wasDeleted.
Change-Id: Ied4b9daa7dc78295b0d36a536839845c4db2fb78
Reviewed-by: MÃ¥rten Nordheim <marten.nordheim@qt.io>
Reviewed-by: Timur Pocheptsov <timur.pocheptsov@qt.io>
Reviewed-by: David Faure <david.faure@kdab.com>
Diffstat (limited to 'src/network/kernel/qdnslookup.cpp')
0 files changed, 0 insertions, 0 deletions