diff options
| author | 2024-11-18 19:08:04 +0000 | |
|---|---|---|
| committer | 2024-11-18 19:08:04 +0000 | |
| commit | ffbd83c6226273ae00781f42e52f485d895ee864 (patch) | |
| tree | 8b56a2d9f818bdff5c274be07abc42f4606ac1d2 /vulkan/libvulkan/api.cpp | |
| parent | 6e56673448d0b83d8ee9128b1c426b1ee58d318d (diff) | |
input: don't log the whole MotionEvent in index checks
We've had a crash report where this causes infinite recursion, probably
through the following call sequence:
operator<<(std::ostream&, const MotionEvent&)
→ MotionEvent::get(X|Y)
→ MotionEvent::getAxisValue
→ MotionEvent::getHistoricalAxisValue
→ MotionEvent::getHistoricalRawPointerCoords
→ operator<<(std::ostream&, const MotionEvent&)
It's unclear how the MotionEvent gets corrupted such that
getHistoricalRawPointerCoords is called with invalid indexes, but the
simple fix is to only log a useful subset of the whole event in these
checks.
Bug: 379368465
Test: m checkinput
Flag: EXEMPT bug fix
Change-Id: I0822f88fc7da6ba08ba6dbbab71ca5aaf78fc35d
Diffstat (limited to 'vulkan/libvulkan/api.cpp')
0 files changed, 0 insertions, 0 deletions