[wayland] Avoid compare to unintialised value in keyboard repeat

Summary:
If we get a key event for which

if (m_xkb->shouldKeyRepeat(key) &&
waylandServer()->seat()->keyRepeatDelay() != 0) fails

m_key will be unitialised and on release we have a compare against
unitialised memory.

Won't do any harm, it'll just stop a timer that isn't running, but
valgrind complains.

0 is the value QKeyEvent uses when nativeScanCode is unknown so a safe
initial values.

Reviewers: #kwin, zzag

Reviewed By: #kwin, zzag

Subscribers: kwin

Tags: #kwin

Differential Revision: https://phabricator.kde.org/D23748
This commit is contained in:
David Edmundson 2019-09-23 22:13:54 +02:00
parent a209ee7865
commit 04845b6007

View file

@ -47,7 +47,7 @@ private:
QTimer *m_timer;
Xkb *m_xkb;
quint32 m_time;
quint32 m_key;
quint32 m_key = 0;
};