diff options
| author | 2024-06-03 20:50:46 +0000 | |
|---|---|---|
| committer | 2024-06-08 00:54:33 +0000 | |
| commit | 9f9f08f4764206a6c424f0399b419e0463c24834 (patch) | |
| tree | fda1ec283f92e44dcb91997974b3e0fd0a4fbe69 /libs/input/PointerController.cpp | |
| parent | 715e4fd95497c629f7c7e0cf4790dd23849797dc (diff) | |
Address memory leak in AppStartInfoTracker
Records were cleared for all error cases, but for success they were only cleared if it reached fully drawn.
This is an issue for 2 reasons:
- fully drawn is not required, so there are success cases where the in prog record is never removed.
- it's just not very robust
To fix this, add logic to ensure that the in progress list doesn't exceed a max size, and that removals are
done of oldest records first.
Also refactors some namings and test details, and adds a new test.
Bug: 342282017
Test: run new test, add logging and confirm correct records are cleared
Flag: EXEMPT - day-to-day bugfix
Change-Id: I2e542e44080cbc9910703a9c3227e344ae4f005f
Diffstat (limited to 'libs/input/PointerController.cpp')
0 files changed, 0 insertions, 0 deletions