diff options
author | Tuomas Tynkkynen <tuomas@tuxera.com> | 2018-02-25T21·51+0200 |
---|---|---|
committer | Tuomas Tynkkynen <tuomas@tuxera.com> | 2018-03-02T15·30+0200 |
commit | a0e38c16bc7773d0fc62771a9935e079c00899ef (patch) | |
tree | 9f30fc3be4c4be6742d220a1b0363be8f053d383 /tests/lang/eval-okay-backslash-newline-1.exp | |
parent | 939cf4ccebb199e61da648873fb078ae8833263f (diff) |
libexpr: Recognize newline in more places in lexer
Flex's regexes have an annoying feature: the dot matches everything except a newline. This causes problems for expressions like: "${0}\ " where the backslash-newline combination matches this rule instead of the intended one mentioned in the comment: <STRING>\$|\\|\$\\ { /* This can only occur when we reach EOF, otherwise the above (...|\$[^\{\"\\]|\\.|\$\\.)+ would have triggered. This is technically invalid, but we leave the problem to the parser who fails with exact location. */ return STR; } However, the parser actually accepts the resulting token sequence ('"' DOLLAR_CURLY 0 '}' STR '"'), which is a problem because the lexer rule didn't assign anything to yylval. Ultimately this leads to a crash when dereferencing a NULL pointer in ExprConcatStrings::bindVars(). The fix does change the syntax of the language in some corner cases but I think it's only turning previously invalid (or crashing) syntax to valid syntax. E.g. "a\ b" and ''a''\ b'' were previously syntax errors but now both result in "a\nb". Found by afl-fuzz.
Diffstat (limited to 'tests/lang/eval-okay-backslash-newline-1.exp')
-rw-r--r-- | tests/lang/eval-okay-backslash-newline-1.exp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/tests/lang/eval-okay-backslash-newline-1.exp b/tests/lang/eval-okay-backslash-newline-1.exp new file mode 100644 index 000000000000..3e754364cc9c --- /dev/null +++ b/tests/lang/eval-okay-backslash-newline-1.exp @@ -0,0 +1 @@ +"a\nb" |