Avoid integer overflow by disallowing Unicode sequences of more than …

…6 digits. See #335.
samhocevar · Jun 18, 2020 · b771c98 · b771c98
1 parent a3286c2
commit b771c98
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/settings/Settings.cs b/src/settings/Settings.cs
@@ -363,8 +363,8 @@ public static bool GetGenericSequenceResult(KeySequence sequence, out string res
             return true;
         }
 
-        private static Regex m_match_gen_prefix = new Regex(@"^[uU][0-9a-fA-F]*$");
-        private static Regex m_match_gen_seq = new Regex(@"^[uU]([0-9a-fA-F]+)( |{return})$");
+        private static Regex m_match_gen_prefix = new Regex(@"^[uU][0-9a-fA-F]{0,6}$");
+        private static Regex m_match_gen_seq = new Regex(@"^[uU]([0-9a-fA-F]{1,6})( |{return})$");
 
         public static List<SequenceDescription> GetSequenceDescriptions() => m_sequences.GetSequenceDescriptions();