1berry/qtbase
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

QDnsLookupRunnable: replace a volatile bool with an atomic int

Browse Source 
Since there is non-atomic data that is protected by 'triedResolve',
the (outer) read from triedResolve needs to have acquire, and the
store needs to have release semantics. The release implied by the
mutex unlock is not good enough because it only synchronises-with
the locking of the same mutex, which not all threads execute.

Change-Id: If46b3ea6ccfdd66ca41ce44d4f45bef2c2c30f72
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
Reviewed-by: Shane Kearns <shane.kearns@accenture.com>
This commit is contained in:
Marc Mutz 2012-08-24 09:47:55 +02:00 committed by Qt by Nokia
parent 462a266edf
commit b2edd830b1
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/network/kernel/qdnslookup_unix.cpp
View File

@ -107,12 +107,12 @@ static void resolveLibrary()
void QDnsLookupRunnable::query(const int requestType, const QByteArray &requestName, QDnsLookupReply *reply)
{
// Load dn_expand, res_ninit and res_nquery on demand.
static volatile bool triedResolve = false;
if (!triedResolve) {
static QBasicAtomicInt triedResolve = Q_BASIC_ATOMIC_INITIALIZER(false);
if (!triedResolve.loadAcquire()) {
QMutexLocker locker(QMutexPool::globalInstanceGet(&local_res_ninit));
if (!triedResolve) {
if (!triedResolve.load()) {
resolveLibrary();
triedResolve = true;
triedResolve.storeRelease(true);
}
}