]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Prevent mis-encoding of "trailing junk after numeric literal" errors.
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 5 Sep 2024 16:42:33 +0000 (12:42 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 5 Sep 2024 16:42:33 +0000 (12:42 -0400)
commitf37ac613a835c8ff28a2f23abe14c88fbac8b039
tree2221dd271d20cc4883744867a79d41c4eb9d4d66
parent777f50b9b5822d1058c59fc2ee1d1a7e9c766bf8
Prevent mis-encoding of "trailing junk after numeric literal" errors.

Since commit 2549f0661, we reject an identifier immediately following
a numeric literal (without separating whitespace), because that risks
ambiguity with hex/octal/binary integers.  However, that patch used
token patterns like "{integer}{ident_start}", which is problematic
because {ident_start} matches only a single byte.  If the first
character after the integer is a multibyte character, this ends up
with flex reporting an error message that includes a partial multibyte
character.  That can cause assorted bad-encoding problems downstream,
both in the report to the client and in the postmaster log file.

To fix, use {identifier} not {ident_start} in the "junk" token
patterns, so that they will match complete multibyte characters.
This seems generally better user experience quite aside from the
encoding problem: for "123abc" the error message will now say that
the error appeared at or near "123abc" instead of "123a".

While at it, add some commentary about why these patterns exist
and how they work.

Report and patch by Karina Litskevich; review by Pavel Borisov.
Back-patch to v15 where the problem came in.

Discussion: https://postgr.es/m/CACiT8iZ_diop=0zJ7zuY3BXegJpkKK1Av-PU7xh0EDYHsa5+=g@mail.gmail.com
src/backend/parser/scan.l
src/fe_utils/psqlscan.l
src/interfaces/ecpg/preproc/pgc.l
src/test/regress/expected/numerology.out