Handle regex syntax error for torrent filtering by showing 0 torrents q…

…bittorrent#20566 (comment)
HamletDuFromage · Aug 8, 2024 · 7688765 · 7688765
1 parent bee56f2
commit 7688765
Showing 1 changed file with 13 additions and 3 deletions.
diff --git a/src/webui/www/private/scripts/dynamicTable.js b/src/webui/www/private/scripts/dynamicTable.js
@@ -1502,9 +1502,19 @@ window.qBittorrent.DynamicTable ??= (() => {
             const rows = this.rows.getValues();
             const useRegex = $("torrentsFilterRegexBox").checked;
             const filterText = $("torrentsFilterInput").value.trim().toLowerCase();
-            const filterTerms = (filterText.length > 0)
-                ? (useRegex ? new RegExp(filterText) : filterText.split(" "))
-                : null;
+            let filterTerms = null;
+            if (filterText.length > 0) {
+                if (useRegex) {
+                    try {
+                        filterTerms = new RegExp(filterText);
+                    } catch (e) {
+                        console.error(`Invalid regex pattern: ${filterText}`);
+                        return filteredRows;
+                    }
+                } else {
+                    filterTerms = filterText.split(" ");
+                }
+            }
 
             for (let i = 0; i < rows.length; ++i) {
                 if (this.applyFilter(rows[i], selected_filter, selected_category, selectedTag, selectedTracker, filterTerms)) {