diff options
| author | Axel Spoerl <axel.spoerl@qt.io> | 2025-11-20 13:03:22 +0100 |
|---|---|---|
| committer | Axel Spoerl <axel.spoerl@qt.io> | 2025-11-25 19:26:23 +0100 |
| commit | 36de22f88787498045f69f493c944c5a6370d091 (patch) | |
| tree | 2babf0767aa48ac369cca5019c26cb0fe5670437 /src/corelib/thread/qbasicatomic.h | |
| parent | 1a0ea9d3481ae8882697a27899d778ab782ee5de (diff) | |
Account for null QPointer in QWidgetAnimator
QWidgetAnimator has a QHash member m_animation_map with all active
animations. Since it doesn't own the animations, they can get deleted
while their pointers are still in the map. To avoid UAF, the
QPropertyAnimation pointers are held in a QPointer.
When a null QPointer was in the map, it caused a nullptr dereference
and animating() returned true, even though nothing was animating.
Add missing nullptr check.
Don't count null QPointer as an ongoing animation.
Amends 33214af3784feacb2d5188bbf07da92f45f582f9.
Fixes: QTBUG-141761
Pick-to: 6.10 6.8 6.5
Change-Id: Iec081472ece2ec86b9ddcb7616ed65401be20cc4
Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io>
Diffstat (limited to 'src/corelib/thread/qbasicatomic.h')
0 files changed, 0 insertions, 0 deletions
