From 571d6151f0983e488feafb3cd6c9c74bbd038082 Mon Sep 17 00:00:00 2001
From: Andrei Karas <akaras@inbox.ru>
Date: Thu, 18 Feb 2016 15:50:43 +0300
Subject: Add some extra checks and logging while sorting keys.

---
 src/input/inputactionsortfunctor.h | 17 +++++++++++++++--
 src/input/inputmanager.cpp         | 21 ++++++++++++++++++++-
 2 files changed, 35 insertions(+), 3 deletions(-)

(limited to 'src')

diff --git a/src/input/inputactionsortfunctor.h b/src/input/inputactionsortfunctor.h
index 6a3a72245..6884f9c11 100644
--- a/src/input/inputactionsortfunctor.h
+++ b/src/input/inputactionsortfunctor.h
@@ -21,6 +21,8 @@
 #ifndef INPUT_INPUTACTIONSORTFUNCTOR_H
 #define INPUT_INPUTACTIONSORTFUNCTOR_H
 
+#include "logger.h"
+
 #include "input/inputactiondata.h"
 
 #include "localconsts.h"
@@ -31,8 +33,19 @@ class InputActionSortFunctor final
         bool operator() (const InputActionT key1,
                          const InputActionT key2) const
         {
-            return keys[CAST_SIZE(key1)].priority
-                >= keys[CAST_SIZE(key2)].priority;
+            const size_t k1 = CAST_SIZE(key1);
+            const size_t k2 = CAST_SIZE(key2);
+            if (k1 >= CAST_SIZE(InputAction::TOTAL))
+            {
+                logger->log("broken key1: %ld", static_cast<long>(k1));
+                return false;
+            }
+            if (k2 >= CAST_SIZE(InputAction::TOTAL))
+            {
+                logger->log("broken key2: %ld", static_cast<long>(k2));
+                return false;
+            }
+            return keys[k1].priority >= keys[k2].priority;
         }
 
         const InputActionData *keys A_NONNULLPOINTER;
diff --git a/src/input/inputmanager.cpp b/src/input/inputmanager.cpp
index 4f9c1ce6b..eb6627d2a 100644
--- a/src/input/inputmanager.cpp
+++ b/src/input/inputmanager.cpp
@@ -921,8 +921,27 @@ void InputManager::updateKeyActionMap(KeyToActionMap &restrict actionMap,
     FOR_EACH (KeyToActionMapIter, it, actionMap)
     {
         KeysVector *const keys = &it->second;
-        if (keys && keys->size() > 1)
+        if (!keys)
+            continue;
+        const size_t sz = keys->size();
+        if (sz > 1)
+        {
+            logger->log("keys size: %d", CAST_S32(sz));
+            int sum = 0;
+            FOR_EACHP (KeysVectorIter, itk, keys)
+            {
+                const size_t val = CAST_SIZE(*itk);
+                if (val >= CAST_SIZE(InputAction::TOTAL))
+                {
+                    logger->log("wrong action index: %ld",
+                        static_cast<long>(val));
+                    break;
+                }
+                sum += inputActionData[val].priority;
+            }
+            logger->log("sum: %d", sum);
             std::sort(keys->begin(), keys->end(), inputActionDataSorter);
+        }
     }
 }
 
-- 
cgit v1.2.3-60-g2f50