diff options
author | Marc Mutz <marc.mutz@kdab.com> | 2019-08-23 09:35:20 +0200 |
---|---|---|
committer | Marc Mutz <marc.mutz@kdab.com> | 2019-08-23 10:39:43 +0200 |
commit | 96ff6e8ebe80215a0d35055c7382bb1cf58fc660 (patch) | |
tree | 9f4bc5af1205d2908c8616bccfc033e8ba14f83a /qmake | |
parent | b1db1dd655788fb685949959c99c901515bfd8ab (diff) |
QGuiApplication: drop mutex before emitting fontChanged()
Emitting a signal executes an unknowable amount of code. We shouldn't
hold a mutex while doing so. E.g., if the signal emission causes
another call to QGuiApplication::setFont(), the old code would
deadlock, since applicationFontMutex is not recursive.
Fix by taking a copy of the application font under mutex protection,
then dropping the lock for the emission of the signal.
Change-Id: Ib2569b3a08af6ef5f38459a19f74cb0db27b7772
Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io>
Reviewed-by: Lars Knoll <lars.knoll@qt.io>
Diffstat (limited to 'qmake')
0 files changed, 0 insertions, 0 deletions