Bugfix: keypress: Only use the lower 8 bits of the mask so that mouse buttons are...

Sometimes, when the mouse button gets stuck, state contains the bit for
BUTTON_MASK_1 (or other buttons). We filter them out to continue processing
keys correctly.

diff --git a/src/handlers.c b/src/handlers.c
index c45be5f46fdb99a19a004d27f4d72fc05e89c823..bc98c86f5c5aad5ee921dd454c61268fe90ebf5b 100644 (file)
--- a/src/handlers.c
+++ b/src/handlers.c
@@ -111,6 +111,10 @@ int handle_key_press(void *ignored, xcb_connection_t *conn, xcb_key_press_event_
         /* Remove the numlock bit, all other bits are modifiers we can bind to */
         uint16_t state_filtered = event->state & ~(xcb_numlock_mask | XCB_MOD_MASK_LOCK);
 
+        /* Only use the lower 8 bits of the state (modifier masks) so that mouse
+         * button masks are filtered out */
+        state_filtered &= 0xFF;
+
         /* Find the binding */
         Binding *bind;
         TAILQ_FOREACH(bind, &bindings, bindings)