Merge pull request #38 from Tom94/regex

Prevent hard-crash when invalid regex syntax is used
Tom94 · Mar 13, 2018 · 859a81e · 859a81e
2 parents b128755 + fc33932
commit 859a81e
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/src/Common.cpp b/src/Common.cpp
@@ -86,8 +86,12 @@ bool matches(string text, string filter, bool isRegex) {
             return true;
         }
 
-        regex searchRegex{filter, std::regex_constants::ECMAScript | std::regex_constants::icase};
-        return regex_search(text, searchRegex);
+        try {
+            regex searchRegex{filter, std::regex_constants::ECMAScript | std::regex_constants::icase};
+            return regex_search(text, searchRegex);
+        } catch (const regex_error&) {
+            return false;
+        }
     };
 
     return isRegex ? matchesRegex(text, filter) : matchesFuzzy(text, filter);