From 0c2338abbb17b7b319f36a73d8db77735346804f Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Mon, 3 Feb 2014 19:47:57 -0500 Subject: [PATCH] Fix lexing of U& sequences just before EOF. Commit a5ff502fceadc7c203b0d7a11b45c73f1b421f69 was a brick shy of a load in the backend lexer too, not just psql. Per further testing of bug #9068. In passing, improve related comments. --- src/backend/parser/scan.l | 17 +++++++++++------ src/bin/psql/psqlscan.l | 3 ++- 2 files changed, 13 insertions(+), 7 deletions(-) diff --git a/src/backend/parser/scan.l b/src/backend/parser/scan.l index 68492204c7..e102e7c81e 100644 --- a/src/backend/parser/scan.l +++ b/src/backend/parser/scan.l @@ -155,6 +155,9 @@ extern void core_yyset_column(int column_no, yyscan_t yyscanner); * quoted string with Unicode escapes * end of a quoted string with Unicode escapes, UESCAPE can follow * Unicode surrogate pair in extended quoted string + * + * Remember to add an <> case whenever you add a new exclusive state! + * The default one is probably not the right thing. */ %x xb @@ -545,12 +548,13 @@ other . {quotefail} { /* throw back all but the quote */ yyless(1); - /* handle possible UESCAPE in xusend mode */ + /* xusend state looks for possible UESCAPE */ BEGIN(xusend); } -{whitespace} +{whitespace} { /* stay in xusend state over whitespace */ } {other} | -{xustop1} { +{xustop1} | +<> { /* no UESCAPE after the quote, throw back everything */ yyless(0); BEGIN(INITIAL); @@ -725,12 +729,13 @@ other . } {dquote} { yyless(1); - /* handle possible UESCAPE in xuiend mode */ + /* xuiend state looks for possible UESCAPE */ BEGIN(xuiend); } -{whitespace} { } +{whitespace} { /* stay in xuiend state over whitespace */ } {other} | -{xustop1} { +{xustop1} | +<> { /* no UESCAPE after the quote, throw back everything */ char *ident; diff --git a/src/bin/psql/psqlscan.l b/src/bin/psql/psqlscan.l index 7d61f932fd..9231459755 100644 --- a/src/bin/psql/psqlscan.l +++ b/src/bin/psql/psqlscan.l @@ -166,7 +166,8 @@ static void escape_variable(bool as_ident); * extended quoted strings (support backslash escape sequences) * $foo$ quoted strings * quoted identifier with Unicode escapes - * end of a quoted identifier with Unicode escapes, UESCAPE can follow * quoted string with Unicode escapes + * end of a quoted identifier with Unicode escapes, UESCAPE can follow + * quoted string with Unicode escapes * end of a quoted string with Unicode escapes, UESCAPE can follow * * Note: we intentionally don't mimic the backend's state; we have