dan/oxc
1
0
Fork 0
mirror of https://github.com/danbulant/oxc synced 2026-05-24 12:21:58 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix(regular_expression): Handle unterminated character class (#5523)

Browse source 
`/[/` is reported by `debug_assert!`, but should not.
This commit is contained in:
leaysgur 2024-09-06 03:28:33 +00:00
parent 10e8984d93
commit 88b7ddb7e0
2 changed files with 8 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
crates/oxc_regular_expression/src/body_parser/mod.rs
View file

@ -171,6 +171,10 @@ mod test {
("(?=a){1}", ParserOptions::default().with_unicode_mode()), ("(?=a){1}", ParserOptions::default().with_unicode_mode()),
("(?!a){1}", ParserOptions::default().with_unicode_mode()), ("(?!a){1}", ParserOptions::default().with_unicode_mode()),
(r"[\d-\D]", ParserOptions::default().with_unicode_mode()), (r"[\d-\D]", ParserOptions::default().with_unicode_mode()),
("[", ParserOptions::default()),
("[", ParserOptions::default().with_unicode_sets_mode()),
("[[", ParserOptions::default().with_unicode_sets_mode()),
("[[]", ParserOptions::default().with_unicode_sets_mode()),
("[z-a]", ParserOptions::default()), ("[z-a]", ParserOptions::default()),
(r"[a-c]]", ParserOptions::default().with_unicode_mode()), (r"[a-c]]", ParserOptions::default().with_unicode_mode()),
( (

5
crates/oxc_regular_expression/src/body_parser/parser.rs
View file

@ -771,7 +771,10 @@ impl<'a> PatternParser<'a> {
&mut self, &mut self,
) -> Result<(ast::CharacterClassContentsKind, Vec<'a, ast::CharacterClassContents<'a>>)> { ) -> Result<(ast::CharacterClassContentsKind, Vec<'a, ast::CharacterClassContents<'a>>)> {
// [empty] // [empty]
if self.reader.peek().filter(|&cp| cp == ']' as u32).is_some() { if self.reader.peek().filter(|&cp| cp == ']' as u32).is_some()
// Unterminated
|| self.reader.peek().is_none()
{
return Ok((ast::CharacterClassContentsKind::Union, Vec::new_in(self.allocator))); return Ok((ast::CharacterClassContentsKind::Union, Vec::new_in(self.allocator)));
} }