[client] Properly track enteredSurface in Keyboard
ClosedPublic

Authored by graesslin on Jul 16 2017, 6:25 PM.

Details

Summary

ASAN found a heap-use-after-free when deleting the focused keyboard
surface in the client library. Keyboard did not track the lifetime of
the focused surface and thus one can access already freed memory.

Test Plan

Adjusted auto test to verify the variable gets cleared

Diff Detail

Repository
R127 KWayland
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
graesslin created this revision.Jul 16 2017, 6:25 PM
Restricted Application added projects: Plasma on Wayland, Frameworks. · View Herald TranscriptJul 16 2017, 6:25 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson accepted this revision.Jul 16 2017, 9:10 PM
This revision is now accepted and ready to land.Jul 16 2017, 9:10 PM
This revision was automatically updated to reflect the committed changes.