Fix a bug in the T34039 hack in case when a modifier key is not mapped.
authorAlexander Gavrilov <angavrilov@gmail.com>
Sat, 16 Nov 2019 08:56:38 +0000 (11:56 +0300)
committerAlexander Gavrilov <angavrilov@gmail.com>
Sat, 16 Nov 2019 08:56:38 +0000 (11:56 +0300)
In order to recover from a transient Focus Out - Focus In disruption
in the middle of a shortcut, which can be caused by certain window
managers, Blender has code that checks which modifier keys are pressed
after Focus In and restores the modifier state based on that.

If one of the Ctrl, Shift, Alt, Super keys is not mapped anywhere
in the active keyboard layout, XKeysymToKeycode returns the invalid
zero keycode, and reading the key state produces garbage, which can
cause an invalid modifier state. Check the return value to avoid this.

intern/ghost/intern/GHOST_SystemX11.cpp

index 1ca412fbccca64d06f48486c9fc286b261227a4d..cb2a04a8a87ec1293d5d6287b72534fd230d4133 100644 (file)
@@ -700,7 +700,7 @@ bool GHOST_SystemX11::processEvents(bool waitForEvent)
 
               for (int i = 0; i < (sizeof(modifiers) / sizeof(*modifiers)); i++) {
                 KeyCode kc = XKeysymToKeycode(m_display, modifiers[i]);
-                if (((xevent.xkeymap.key_vector[kc >> 3] >> (kc & 7)) & 1) != 0) {
+                if (kc != 0 && ((xevent.xkeymap.key_vector[kc >> 3] >> (kc & 7)) & 1) != 0) {
                   pushEvent(new GHOST_EventKey(getMilliSeconds(),
                                                GHOST_kEventKeyDown,
                                                window,