diff options
author | Robert Griebl <robert.griebl@qt.io> | 2020-11-18 23:12:35 +0100 |
---|---|---|
committer | Robert Griebl <robert.griebl@qt.io> | 2020-11-19 12:28:44 +0100 |
commit | 2fab1971fed22a9aa852e8df8155b6d5da83279d (patch) | |
tree | 3f23465f66a7c81b71c0968c00cdd034ab72776e /src/network/access | |
parent | 1c1c1e4559db204f339c26bd6cbb5d568dcd47e2 (diff) |
Fix memory corruption in QDBusInterface signal emissions
If more than one signal parameter required conversions (e.g. 2
QVariantMaps), then the auxParameter list would be reallocated on the
second append. This resulted in the reference to the first conversion
(stored in params) to be broken.
Found with valgrind after the QtApplicationManager started crashing
weirdly when built against Qt 6. The same code is in Qt 5, but it
just works fine there: I guess the reallocation strategy in QList is
different there, so we never have to reallocate the list.
Change-Id: I2e0c8906ebc9474c4ec9f53cafc1689003d5c4c5
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
Diffstat (limited to 'src/network/access')
0 files changed, 0 insertions, 0 deletions