]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Fix JSON error reporting for many cases of erroneous string values.
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 13 Mar 2023 19:19:00 +0000 (15:19 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 13 Mar 2023 19:19:00 +0000 (15:19 -0400)
commit74a1a36d755bf5a5c656d78ef7f1df1cfeeeeb20
treeb3a328ff78bb73dbbf510540ae89a14be648d9ef
parent5fd61bdc114f85ce57da1f139c8bda0f41d1951b
Fix JSON error reporting for many cases of erroneous string values.

The majority of error exit cases in json_lex_string() failed to
set lex->token_terminator, causing problems for the error context
reporting code: it would see token_terminator less than token_start
and do something more or less nuts.  In v14 and up the end result
could be as bad as a crash in report_json_context().  Older
versions accidentally avoided that fate; but all versions produce
error context lines that are far less useful than intended,
because they'd stop at the end of the prior token instead of
continuing to where the actually-bad input is.

To fix, invent some macros that make it less notationally painful
to do the right thing.  Also add documentation about what the
function is actually required to do; and in >= v14, add an assertion
in report_json_context about token_terminator being sufficiently
far advanced.

Per report from Nikolay Shaplov.  Back-patch to all supported
versions.

Discussion: https://postgr.es/m/7332649.x5DLKWyVIX@thinkpad-pgpro
src/backend/utils/adt/jsonfuncs.c
src/common/jsonapi.c
src/test/regress/expected/json_encoding.out
src/test/regress/expected/json_encoding_1.out