summaryrefslogtreecommitdiffstats
path: root/src/plugins/sqldrivers/sqlite/qsql_sqlite.cpp
diff options
context:
space:
mode:
authorAllan Sandfeld Jensen <allan.jensen@qt.io>2022-04-12 16:25:08 +0200
committerAllan Sandfeld Jensen <allan.jensen@qt.io>2022-04-15 10:27:52 +0200
commit830b1550de303dd59c29a87c28e44fa41112b8f4 (patch)
tree0e856f32f6fb1b1068f9649800d54e1ad5bc357a /src/plugins/sqldrivers/sqlite/qsql_sqlite.cpp
parent8b3f2dd994619fd224f54e59b7803d3e7d5896b6 (diff)
Fix race condition in futex-based QSemaphore
Add one and reset the wakeAll bit atomically. This avoids a race in a case where an acquiring thread is owning the semaphore, and deleting it after a set number of releases (one for each thread referencing the semaphore). Two releasing threads could enter the if statement under futexNeedsWake(prevValue), the counter been incremented at this point and reached the value being acquired, meaning the thread acquiring can be awakened by just one of the two releasers, delete the semaphore, and then the second releaser would access the now deleted semaphore. The patch avoids that by unsetting and reading the wakeAll bit atomically, so only one thread will try to wake all threads. Pick-to: 6.3 6.2 5.15 Fixes: QTBUG-102484 Change-Id: I32172ed44d74378c627918e19b9e1aaadb5c6d1d Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
Diffstat (limited to 'src/plugins/sqldrivers/sqlite/qsql_sqlite.cpp')
0 files changed, 0 insertions, 0 deletions