diff options
author | Bradley T. Hughes <bradley.hughes@nokia.com> | 2010-08-09 11:27:31 (GMT) |
---|---|---|
committer | Bradley T. Hughes <bradley.hughes@nokia.com> | 2010-08-09 12:57:48 (GMT) |
commit | 3b9c811a658d45f6e84a98ac26a5d122b34254fc (patch) | |
tree | 19bc8553629812d947353b724a83a3ac7cf4a28f /tests/benchmarks | |
parent | 163458f6363d32be453a186e02216f574f62dda3 (diff) | |
download | Qt-3b9c811a658d45f6e84a98ac26a5d122b34254fc.zip Qt-3b9c811a658d45f6e84a98ac26a5d122b34254fc.tar.gz Qt-3b9c811a658d45f6e84a98ac26a5d122b34254fc.tar.bz2 |
Fix invalid memory write during recursive timer activation
The handler for one timer recurses the event loop, and the handler for
another timer removes the first timer, returning from the recursion and
the handler for the first timer causes an invalid write (since the timer
info for the first timer has been deleted).
Fix this by keeping an active reference in QTimerInfo (instead of just a
bool inTimerEvent). If this is non-zero, the timer is currently being
delivered, so we prevent more delivery. When a timer is removed and it's
activateRef is set, we clear it so that the delivery code knows now to
write to memory that's already been freed.
A side effect of this change is that we no longer need to track the
currentTimerInfo "globally" anymore, it can be a normal local variable
in the QTimerInfoList::activateTimers() function.
Task-number: QT-3553
Reviewed-by: olivier
Reviewed-by: joao
Diffstat (limited to 'tests/benchmarks')
0 files changed, 0 insertions, 0 deletions