diff options
author | Thiago Macieira <thiago.macieira@intel.com> | 2015-04-03 22:21:08 -0700 |
---|---|---|
committer | Thiago Macieira <thiago.macieira@intel.com> | 2015-04-15 00:21:57 +0000 |
commit | d370878aa0510e1e51eb9014965f505e395f3f81 (patch) | |
tree | ffdfd3c5b95f32fece7bbbe0febc9ba91d4bb6b6 /mkspecs/winphone-arm-msvc2013/qplatformdefs.h | |
parent | 2bcbe227bdcfded9ae01b289e78aa9e117613056 (diff) |
Fix undefined behavior: accessing an object after destruction
QDBusConnectionPrivate keeps a pointer to a QDBusConnectionInterface,
which in turn holds a reference back to that QDBusConnectionPrivate.
During the interface object's destruction, the QDBusAbstractInterface's
destructor checks if the interface is still valid. That access is
undefined behavior, but has so far been benign in all our uses since
the memory had not yet been freed (just destroyed) and the reference
count went from 0 to -1.
To be on the safe side, we destroy now the QDBusConnectionInterface
object while the Private is still valid. It will bring the reference
count down from 0 to a negative number, but won't cause any other
effects.
Change-Id: I9a75ad8521ae4e5cbbe5ffff13d1b80057e13809
Reviewed-by: Alex Blasche <alexander.blasche@theqtcompany.com>
Diffstat (limited to 'mkspecs/winphone-arm-msvc2013/qplatformdefs.h')
0 files changed, 0 insertions, 0 deletions