From 467977f20326f704b259157652ee9b7ba2efaa78 Mon Sep 17 00:00:00 2001 From: Guillaume Maudoux Date: Thu, 2 Jul 2015 23:53:04 +0200 Subject: Fix the parsing of "$"'s in strings. --- src/libexpr/lexer.l | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) (limited to 'src/libexpr/lexer.l') diff --git a/src/libexpr/lexer.l b/src/libexpr/lexer.l index 7483e5cc4acb..1f2957ec71ad 100644 --- a/src/libexpr/lexer.l +++ b/src/libexpr/lexer.l @@ -129,12 +129,11 @@ or { return OR_KW; } \} { POP_STATE(); return '}'; } \" { PUSH_STATE(STRING); return '"'; } -([^\$\"\\]|\$[^\{\"]|\\.)+ { - /* !!! Not quite right: we want a follow restriction on - "$", it shouldn't be followed by a "{". Right now - "$\"" will be consumed as part of a string, rather - than a "$" followed by the string terminator. - Disallow "$\"" for now. */ +([^\$\"\\]|\$[^\{\"\\]|\\.|\$\\.)*\$/\" | +([^\$\"\\]|\$[^\{\"\\]|\\.|\$\\.)+ { + /* It is impossible to match strings ending with '$' with one + regex because trailing contexts are only valid at the end + of a rule. (A sane but undocumented limitation.) */ yylval->e = unescapeStr(data->symbols, yytext); return STR; } -- cgit 1.4.1