Skip to content

Commit

Permalink
syntax: Fix complexity of string parsing. Closes #16624.
Browse files Browse the repository at this point in the history
  • Loading branch information
brson committed Aug 21, 2014
1 parent f92015f commit bc7a85d
Show file tree
Hide file tree
Showing 2 changed files with 31 additions and 5 deletions.
21 changes: 16 additions & 5 deletions src/libsyntax/parse/mod.rs
Expand Up @@ -412,14 +412,21 @@ pub fn str_lit(lit: &str) -> String {
loop {
match chars.next() {
Some((i, c)) => {
let em = error(i);
match c {
'\\' => {
if chars.peek().expect(em.as_slice()).val1() == '\n' {
let ch = chars.peek().unwrap_or_else(|| {
fail!("{}", error(i).as_slice())
}).val1();

if ch == '\n' {
eat(&mut chars);
} else if chars.peek().expect(em.as_slice()).val1() == '\r' {
} else if ch == '\r' {
chars.next();
if chars.peek().expect(em.as_slice()).val1() != '\n' {
let ch = chars.peek().unwrap_or_else(|| {
fail!("{}", error(i).as_slice())
}).val1();

if ch != '\n' {
fail!("lexer accepted bare CR");
}
eat(&mut chars);
Expand All @@ -433,7 +440,11 @@ pub fn str_lit(lit: &str) -> String {
}
},
'\r' => {
if chars.peek().expect(em.as_slice()).val1() != '\n' {
let ch = chars.peek().unwrap_or_else(|| {
fail!("{}", error(i).as_slice())
}).val1();

if ch != '\n' {
fail!("lexer accepted bare CR");
}
chars.next();
Expand Down
15 changes: 15 additions & 0 deletions src/test/run-pass/slowparse-string.rs

Large diffs are not rendered by default.

0 comments on commit bc7a85d

Please sign in to comment.