feat: add note for clarifying regexp error

VirusTotal · Apr 3, 2024 · cf8325b · cf8325b
1 parent 6233989
commit cf8325b
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 1 deletion.
diff --git a/lib/src/compiler/tests/testdata/errors/123.in b/lib/src/compiler/tests/testdata/errors/123.in
@@ -0,0 +1,6 @@
+rule test {
+  strings:
+    $a = /{(foo|bar)/
+  condition:
+  	$a
+}
diff --git a/lib/src/compiler/tests/testdata/errors/123.out b/lib/src/compiler/tests/testdata/errors/123.out
@@ -0,0 +1,7 @@
+error: invalid regular expression
+ --> line:3:11
+  |
+3 |     $a = /{(foo|bar)/
+  |           ^ repetition operator missing expression
+  |
+  = note: did you mean `\{` instead of `{`?
diff --git a/lib/src/re/parser.rs b/lib/src/re/parser.rs
@@ -130,7 +130,8 @@ impl Parser {
                         esc_seq, esc_seq
                     ))
                 }
-                ErrorKind::RepetitionCountUnclosed
+                ErrorKind::RepetitionMissing
+                | ErrorKind::RepetitionCountUnclosed
                 | ErrorKind::RepetitionCountDecimalEmpty => {
                     Some("did you mean `\\{` instead of `{`?".to_string())
                 }