core: Fix out-of-bounds access by TICK event #487

The error is caused in layer_swtich_get_action() and fails to access layer_pressed[][]

diff --git a/tmk_core/common/action.c b/tmk_core/common/action.c
index 05aa12eee87af2631f6da268e1df0d253b390fec..305816d4f0ccbcaad729f87663767415b527741f 100644 (file)
--- a/tmk_core/common/action.c
+++ b/tmk_core/common/action.c
@@ -531,6 +531,8 @@ void clear_keyboard_but_mods(void)
 
 bool is_tap_key(keyevent_t event)
 {
+    if (IS_NOEVENT(event)) { return false; }
+
     action_t action = layer_switch_get_action(event);
 
     switch (action.kind.id) {

diff --git a/tmk_core/common/action_layer.c b/tmk_core/common/action_layer.c
index 31d659b7aab2d6544dd50f535af5dfe05ce1e370..be316697190cc7b2a1533ca606a570872256967d 100644 (file)
--- a/tmk_core/common/action_layer.c
+++ b/tmk_core/common/action_layer.c
@@ -148,6 +148,8 @@ static uint8_t layer_pressed[MATRIX_ROWS][MATRIX_COLS] = {};
 #endif
 action_t layer_switch_get_action(keyevent_t event)
 {
+    if (IS_NOEVENT(event)) return (action_t)ACTION_NO;
+
     uint8_t layer = 0;
 #ifndef NO_TRACK_KEY_PRESS
     if (event.pressed) {