NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitc25a929

committed

Fix JSON error reporting for many cases of erroneous string values.

The majority of error exit cases in json_lex_string() failed toset lex->token_terminator, causing problems for the error contextreporting code: it would see token_terminator less than token_startand do something more or less nuts. In v14 and up the end resultcould be as bad as a crash in report_json_context(). Olderversions accidentally avoided that fate; but all versions produceerror context lines that are far less useful than intended,because they'd stop at the end of the prior token instead ofcontinuing to where the actually-bad input is.To fix, invent some macros that make it less notationally painfulto do the right thing. Also add documentation about what thefunction is actually required to do; and in >= v14, add an assertionin report_json_context about token_terminator being sufficientlyfar advanced.Per report from Nikolay Shaplov. Back-patch to all supportedversions.Discussion:https://postgr.es/m/7332649.x5DLKWyVIX@thinkpad-pgpro

1 parent62a91a1 commitc25a929Copy full SHA for c25a929

File tree

3 files changed

+57

-32

lines changed

src
- backend/utils/adt
  - json.c
- test/regress/expected
  - json_encoding.out
  - json_encoding_1.out

3 files changed

+57

-32

lines changed

`‎src/backend/utils/adt/json.c`

Lines changed: 27 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -750,6 +750,13 @@ json_lex(JsonLexContext *lex)`
`750`	`750`
`751`	`751`	`/*`
`752`	`752`	`* The next token in the input stream is known to be a string; lex it.`
	`753`	`+ *`
	`754`	`+ * If lex->strval isn't NULL, fill it with the decoded string.`
	`755`	`+ * Set lex->token_terminator to the end of the decoded input, and in`
	`756`	`+ * success cases, transfer its previous value to lex->prev_token_terminator.`
	`757`	`+ *`
	`758`	`+ * Note: be careful that all error cases advance lex->token_terminator`
	`759`	`+ * to the point after the character we detected the error on.`
`753`	`760`	`*/`
`754`	`761`	`staticinlinevoid`
`755`	`762`	`json_lex_string(JsonLexContext*lex)`
`@@ -837,33 +844,42 @@ json_lex_string(JsonLexContext *lex)`
`837`	`844`	`if (ch >=0xd800&&ch <=0xdbff)`
`838`	`845`	`{`
`839`	`846`	`if (hi_surrogate!=-1)`
	`847`	`+{`
	`848`	`+lex->token_terminator=s+pg_mblen(s);`
`840`	`849`	`ereport(ERROR,`
`841`	`850`	`(errcode(ERRCODE_INVALID_TEXT_REPRESENTATION),`
`842`	`851`	`errmsg("invalid input syntax for type %s",`
`843`	`852`	`"json"),`
`844`	`853`	`errdetail("Unicode high surrogate must not follow a high surrogate."),`
`845`	`854`	`report_json_context(lex)));`
	`855`	`+}`
`846`	`856`	`hi_surrogate= (ch&0x3ff) <<10;`
`847`	`857`	`continue;`
`848`	`858`	`}`
`849`	`859`	`elseif (ch >=0xdc00&&ch <=0xdfff)`
`850`	`860`	`{`
`851`	`861`	`if (hi_surrogate==-1)`
	`862`	`+{`
	`863`	`+lex->token_terminator=s+pg_mblen(s);`
`852`	`864`	`ereport(ERROR,`
`853`	`865`	`(errcode(ERRCODE_INVALID_TEXT_REPRESENTATION),`
`854`	`866`	`errmsg("invalid input syntax for type %s","json"),`
`855`	`867`	`errdetail("Unicode low surrogate must follow a high surrogate."),`
`856`	`868`	`report_json_context(lex)));`
	`869`	`+}`
`857`	`870`	`ch=0x10000+hi_surrogate+ (ch&0x3ff);`
`858`	`871`	`hi_surrogate=-1;`
`859`	`872`	`}`
`860`	`873`
`861`	`874`	`if (hi_surrogate!=-1)`
	`875`	`+{`
	`876`	`+lex->token_terminator=s+pg_mblen(s);`
`862`	`877`	`ereport(ERROR,`
`863`	`878`	`(errcode(ERRCODE_INVALID_TEXT_REPRESENTATION),`
`864`	`879`	`errmsg("invalid input syntax for type %s","json"),`
`865`	`880`	`errdetail("Unicode low surrogate must follow a high surrogate."),`
`866`	`881`	`report_json_context(lex)));`
	`882`	`+}`
`867`	`883`
`868`	`884`	`/*`
`869`	`885`	`* For UTF8, replace the escape sequence by the actual`
`@@ -875,6 +891,7 @@ json_lex_string(JsonLexContext *lex)`
`875`	`891`	`if (ch==0)`
`876`	`892`	`{`
`877`	`893`	`/* We can't allow this, since our TEXT type doesn't */`
	`894`	`+lex->token_terminator=s+pg_mblen(s);`
`878`	`895`	`ereport(ERROR,`
`879`	`896`	`(errcode(ERRCODE_UNTRANSLATABLE_CHARACTER),`
`880`	`897`	`errmsg("unsupported Unicode escape sequence"),`
`@@ -898,24 +915,27 @@ json_lex_string(JsonLexContext *lex)`
`898`	`915`	`}`
`899`	`916`	`else`
`900`	`917`	`{`
	`918`	`+lex->token_terminator=s+pg_mblen(s);`
`901`	`919`	`ereport(ERROR,`
`902`	`920`	`(errcode(ERRCODE_UNTRANSLATABLE_CHARACTER),`
`903`	`921`	`errmsg("unsupported Unicode escape sequence"),`
`904`	`922`	`errdetail("Unicode escape values cannot be used for code point values above 007F when the server encoding is not UTF8."),`
`905`	`923`	`report_json_context(lex)));`
`906`	`924`	`}`
`907`		`-`
`908`	`925`	`}`
`909`	`926`	`}`
`910`	`927`	`elseif (lex->strval!=NULL)`
`911`	`928`	`{`
`912`	`929`	`if (hi_surrogate!=-1)`
	`930`	`+{`
	`931`	`+lex->token_terminator=s+pg_mblen(s);`
`913`	`932`	`ereport(ERROR,`
`914`	`933`	`(errcode(ERRCODE_INVALID_TEXT_REPRESENTATION),`
`915`	`934`	`errmsg("invalid input syntax for type %s",`
`916`	`935`	`"json"),`
`917`	`936`	`errdetail("Unicode low surrogate must follow a high surrogate."),`
`918`	`937`	`report_json_context(lex)));`
	`938`	`+}`
`919`	`939`
`920`	`940`	`switch (*s)`
`921`	`941`	`{`
`@@ -968,28 +988,33 @@ json_lex_string(JsonLexContext *lex)`
`968`	`988`	`extract_mb_char(s)),`
`969`	`989`	`report_json_context(lex)));`
`970`	`990`	`}`
`971`		`-`
`972`	`991`	`}`
`973`	`992`	`elseif (lex->strval!=NULL)`
`974`	`993`	`{`
`975`	`994`	`if (hi_surrogate!=-1)`
	`995`	`+{`
	`996`	`+lex->token_terminator=s+pg_mblen(s);`
`976`	`997`	`ereport(ERROR,`
`977`	`998`	`(errcode(ERRCODE_INVALID_TEXT_REPRESENTATION),`
`978`	`999`	`errmsg("invalid input syntax for type %s","json"),`
`979`	`1000`	`errdetail("Unicode low surrogate must follow a high surrogate."),`
`980`	`1001`	`report_json_context(lex)));`
	`1002`	`+}`
`981`	`1003`
`982`	`1004`	`appendStringInfoChar(lex->strval,*s);`
`983`	`1005`	`}`
`984`	`1006`
`985`	`1007`	`}`
`986`	`1008`
`987`	`1009`	`if (hi_surrogate!=-1)`
	`1010`	`+{`
	`1011`	`+lex->token_terminator=s+pg_mblen(s);`
`988`	`1012`	`ereport(ERROR,`
`989`	`1013`	`(errcode(ERRCODE_INVALID_TEXT_REPRESENTATION),`
`990`	`1014`	`errmsg("invalid input syntax for type %s","json"),`
`991`	`1015`	`errdetail("Unicode low surrogate must follow a high surrogate."),`
`992`	`1016`	`report_json_context(lex)));`
	`1017`	`+}`
`993`	`1018`
`994`	`1019`	`/* Hooray, we found the end of the string! */`
`995`	`1020`	`lex->prev_token_terminator=lex->token_terminator;`

`‎src/test/regress/expected/json_encoding.out`

Lines changed: 12 additions & 12 deletions

Original file line number	Diff line number	Diff line change
`@@ -41,19 +41,19 @@ select json '{ "a": "\ud83d\ude04\ud83d\udc36" }' -> 'a' as correct_in_utf8;`
`41`	`41`	`select json '{ "a": "\ud83d\ud83d" }' -> 'a'; -- 2 high surrogates in a row`
`42`	`42`	`ERROR: invalid input syntax for type json`
`43`	`43`	`DETAIL: Unicode high surrogate must not follow a high surrogate.`
`44`		`-CONTEXT: JSON data, line 1: { "a":...`
	`44`	`+CONTEXT: JSON data, line 1: { "a": "\ud83d\ud83d...`
`45`	`45`	`select json '{ "a": "\ude04\ud83d" }' -> 'a'; -- surrogates in wrong order`
`46`	`46`	`ERROR: invalid input syntax for type json`
`47`	`47`	`DETAIL: Unicode low surrogate must follow a high surrogate.`
`48`		`-CONTEXT: JSON data, line 1: { "a":...`
	`48`	`+CONTEXT: JSON data, line 1: { "a": "\ude04...`
`49`	`49`	`select json '{ "a": "\ud83dX" }' -> 'a'; -- orphan high surrogate`
`50`	`50`	`ERROR: invalid input syntax for type json`
`51`	`51`	`DETAIL: Unicode low surrogate must follow a high surrogate.`
`52`		`-CONTEXT: JSON data, line 1: { "a":...`
	`52`	`+CONTEXT: JSON data, line 1: { "a": "\ud83dX...`
`53`	`53`	`select json '{ "a": "\ude04X" }' -> 'a'; -- orphan low surrogate`
`54`	`54`	`ERROR: invalid input syntax for type json`
`55`	`55`	`DETAIL: Unicode low surrogate must follow a high surrogate.`
`56`		`-CONTEXT: JSON data, line 1: { "a":...`
	`56`	`+CONTEXT: JSON data, line 1: { "a": "\ude04...`
`57`	`57`	`--handling of simple unicode escapes`
`58`	`58`	`select json '{ "a": "the Copyright \u00a9 sign" }' as correct_in_utf8;`
`59`	`59`	`correct_in_utf8`
`@@ -106,7 +106,7 @@ select json '{ "a": "dollar \\u0024 character" }' ->> 'a' as not_an_escape;`
`106`	`106`	`select json '{ "a": "null \u0000 escape" }' ->> 'a' as fails;`
`107`	`107`	`ERROR: unsupported Unicode escape sequence`
`108`	`108`	`DETAIL: \u0000 cannot be converted to text.`
`109`		`-CONTEXT: JSON data, line 1: { "a":...`
	`109`	`+CONTEXT: JSON data, line 1: { "a": "null \u0000...`
`110`	`110`	`select json '{ "a": "null \\u0000 escape" }' ->> 'a' as not_an_escape;`
`111`	`111`	`not_an_escape`
`112`	`112`	`--------------------`
`@@ -144,7 +144,7 @@ ERROR: unsupported Unicode escape sequence`
`144`	`144`	`LINE 1: SELECT '"\u0000"'::jsonb;`
`145`	`145`	`^`
`146`	`146`	`DETAIL: \u0000 cannot be converted to text.`
`147`		`-CONTEXT: JSON data, line 1: ...`
	`147`	`+CONTEXT: JSON data, line 1:"\u0000...`
`148`	`148`	`-- use octet_length here so we don't get an odd unicode char in the`
`149`	`149`	`-- output`
`150`	`150`	`SELECT octet_length('"\uaBcD"'::jsonb::text); -- OK, uppercase and lower case both OK`
`@@ -165,25 +165,25 @@ ERROR: invalid input syntax for type json`
`165`	`165`	`LINE 1: SELECT jsonb '{ "a": "\ud83d\ud83d" }' -> 'a';`
`166`	`166`	`^`
`167`	`167`	`DETAIL: Unicode high surrogate must not follow a high surrogate.`
`168`		`-CONTEXT: JSON data, line 1: { "a":...`
	`168`	`+CONTEXT: JSON data, line 1: { "a": "\ud83d\ud83d...`
`169`	`169`	`SELECT jsonb '{ "a": "\ude04\ud83d" }' -> 'a'; -- surrogates in wrong order`
`170`	`170`	`ERROR: invalid input syntax for type json`
`171`	`171`	`LINE 1: SELECT jsonb '{ "a": "\ude04\ud83d" }' -> 'a';`
`172`	`172`	`^`
`173`	`173`	`DETAIL: Unicode low surrogate must follow a high surrogate.`
`174`		`-CONTEXT: JSON data, line 1: { "a":...`
	`174`	`+CONTEXT: JSON data, line 1: { "a": "\ude04...`
`175`	`175`	`SELECT jsonb '{ "a": "\ud83dX" }' -> 'a'; -- orphan high surrogate`
`176`	`176`	`ERROR: invalid input syntax for type json`
`177`	`177`	`LINE 1: SELECT jsonb '{ "a": "\ud83dX" }' -> 'a';`
`178`	`178`	`^`
`179`	`179`	`DETAIL: Unicode low surrogate must follow a high surrogate.`
`180`		`-CONTEXT: JSON data, line 1: { "a":...`
	`180`	`+CONTEXT: JSON data, line 1: { "a": "\ud83dX...`
`181`	`181`	`SELECT jsonb '{ "a": "\ude04X" }' -> 'a'; -- orphan low surrogate`
`182`	`182`	`ERROR: invalid input syntax for type json`
`183`	`183`	`LINE 1: SELECT jsonb '{ "a": "\ude04X" }' -> 'a';`
`184`	`184`	`^`
`185`	`185`	`DETAIL: Unicode low surrogate must follow a high surrogate.`
`186`		`-CONTEXT: JSON data, line 1: { "a":...`
	`186`	`+CONTEXT: JSON data, line 1: { "a": "\ude04...`
`187`	`187`	`-- handling of simple unicode escapes`
`188`	`188`	`SELECT jsonb '{ "a": "the Copyright \u00a9 sign" }' as correct_in_utf8;`
`189`	`189`	`correct_in_utf8`
`@@ -208,7 +208,7 @@ ERROR: unsupported Unicode escape sequence`
`208`	`208`	`LINE 1: SELECT jsonb '{ "a": "null \u0000 escape" }' as fails;`
`209`	`209`	`^`
`210`	`210`	`DETAIL: \u0000 cannot be converted to text.`
`211`		`-CONTEXT: JSON data, line 1: { "a":...`
	`211`	`+CONTEXT: JSON data, line 1: { "a": "null \u0000...`
`212`	`212`	`SELECT jsonb '{ "a": "null \\u0000 escape" }' as not_an_escape;`
`213`	`213`	`not_an_escape`
`214`	`214`	`------------------------------`
`@@ -238,7 +238,7 @@ ERROR: unsupported Unicode escape sequence`
`238`	`238`	`LINE 1: SELECT jsonb '{ "a": "null \u0000 escape" }' ->> 'a' as fai...`
`239`	`239`	`^`
`240`	`240`	`DETAIL: \u0000 cannot be converted to text.`
`241`		`-CONTEXT: JSON data, line 1: { "a":...`
	`241`	`+CONTEXT: JSON data, line 1: { "a": "null \u0000...`
`242`	`242`	`SELECT jsonb '{ "a": "null \\u0000 escape" }' ->> 'a' as not_an_escape;`
`243`	`243`	`not_an_escape`
`244`	`244`	`--------------------`

`‎src/test/regress/expected/json_encoding_1.out`

Lines changed: 18 additions & 18 deletions

Original file line number	Diff line number	Diff line change
`@@ -35,23 +35,23 @@ SELECT '"\uaBcD"'::json;-- OK, uppercase and lower case both OK`
`35`	`35`	`select json '{ "a": "\ud83d\ude04\ud83d\udc36" }' -> 'a' as correct_in_utf8;`
`36`	`36`	`ERROR: unsupported Unicode escape sequence`
`37`	`37`	`DETAIL: Unicode escape values cannot be used for code point values above 007F when the server encoding is not UTF8.`
`38`		`-CONTEXT: JSON data, line 1: { "a":...`
	`38`	`+CONTEXT: JSON data, line 1: { "a": "\ud83d\ude04...`
`39`	`39`	`select json '{ "a": "\ud83d\ud83d" }' -> 'a'; -- 2 high surrogates in a row`
`40`	`40`	`ERROR: invalid input syntax for type json`
`41`	`41`	`DETAIL: Unicode high surrogate must not follow a high surrogate.`
`42`		`-CONTEXT: JSON data, line 1: { "a":...`
	`42`	`+CONTEXT: JSON data, line 1: { "a": "\ud83d\ud83d...`
`43`	`43`	`select json '{ "a": "\ude04\ud83d" }' -> 'a'; -- surrogates in wrong order`
`44`	`44`	`ERROR: invalid input syntax for type json`
`45`	`45`	`DETAIL: Unicode low surrogate must follow a high surrogate.`
`46`		`-CONTEXT: JSON data, line 1: { "a":...`
	`46`	`+CONTEXT: JSON data, line 1: { "a": "\ude04...`
`47`	`47`	`select json '{ "a": "\ud83dX" }' -> 'a'; -- orphan high surrogate`
`48`	`48`	`ERROR: invalid input syntax for type json`
`49`	`49`	`DETAIL: Unicode low surrogate must follow a high surrogate.`
`50`		`-CONTEXT: JSON data, line 1: { "a":...`
	`50`	`+CONTEXT: JSON data, line 1: { "a": "\ud83dX...`
`51`	`51`	`select json '{ "a": "\ude04X" }' -> 'a'; -- orphan low surrogate`
`52`	`52`	`ERROR: invalid input syntax for type json`
`53`	`53`	`DETAIL: Unicode low surrogate must follow a high surrogate.`
`54`		`-CONTEXT: JSON data, line 1: { "a":...`
	`54`	`+CONTEXT: JSON data, line 1: { "a": "\ude04...`
`55`	`55`	`--handling of simple unicode escapes`
`56`	`56`	`select json '{ "a": "the Copyright \u00a9 sign" }' as correct_in_utf8;`
`57`	`57`	`correct_in_utf8`
`@@ -86,7 +86,7 @@ select json '{ "a": "null \\u0000 escape" }' as not_an_escape;`
`86`	`86`	`select json '{ "a": "the Copyright \u00a9 sign" }' ->> 'a' as correct_in_utf8;`
`87`	`87`	`ERROR: unsupported Unicode escape sequence`
`88`	`88`	`DETAIL: Unicode escape values cannot be used for code point values above 007F when the server encoding is not UTF8.`
`89`		`-CONTEXT: JSON data, line 1: { "a":...`
	`89`	`+CONTEXT: JSON data, line 1: { "a": "the Copyright \u00a9...`
`90`	`90`	`select json '{ "a": "dollar \u0024 character" }' ->> 'a' as correct_everywhere;`
`91`	`91`	`correct_everywhere`
`92`	`92`	`--------------------`
`@@ -102,7 +102,7 @@ select json '{ "a": "dollar \\u0024 character" }' ->> 'a' as not_an_escape;`
`102`	`102`	`select json '{ "a": "null \u0000 escape" }' ->> 'a' as fails;`
`103`	`103`	`ERROR: unsupported Unicode escape sequence`
`104`	`104`	`DETAIL: \u0000 cannot be converted to text.`
`105`		`-CONTEXT: JSON data, line 1: { "a":...`
	`105`	`+CONTEXT: JSON data, line 1: { "a": "null \u0000...`
`106`	`106`	`select json '{ "a": "null \\u0000 escape" }' ->> 'a' as not_an_escape;`
`107`	`107`	`not_an_escape`
`108`	`108`	`--------------------`
`@@ -140,53 +140,53 @@ ERROR: unsupported Unicode escape sequence`
`140`	`140`	`LINE 1: SELECT '"\u0000"'::jsonb;`
`141`	`141`	`^`
`142`	`142`	`DETAIL: \u0000 cannot be converted to text.`
`143`		`-CONTEXT: JSON data, line 1: ...`
	`143`	`+CONTEXT: JSON data, line 1:"\u0000...`
`144`	`144`	`-- use octet_length here so we don't get an odd unicode char in the`
`145`	`145`	`-- output`
`146`	`146`	`SELECT octet_length('"\uaBcD"'::jsonb::text); -- OK, uppercase and lower case both OK`
`147`	`147`	`ERROR: unsupported Unicode escape sequence`
`148`	`148`	`LINE 1: SELECT octet_length('"\uaBcD"'::jsonb::text);`
`149`	`149`	`^`
`150`	`150`	`DETAIL: Unicode escape values cannot be used for code point values above 007F when the server encoding is not UTF8.`
`151`		`-CONTEXT: JSON data, line 1: ...`
	`151`	`+CONTEXT: JSON data, line 1:"\uaBcD...`
`152`	`152`	`-- handling of unicode surrogate pairs`
`153`	`153`	`SELECT octet_length((jsonb '{ "a": "\ud83d\ude04\ud83d\udc36" }' -> 'a')::text) AS correct_in_utf8;`
`154`	`154`	`ERROR: unsupported Unicode escape sequence`
`155`	`155`	`LINE 1: SELECT octet_length((jsonb '{ "a": "\ud83d\ude04\ud83d\udc3...`
`156`	`156`	`^`
`157`	`157`	`DETAIL: Unicode escape values cannot be used for code point values above 007F when the server encoding is not UTF8.`
`158`		`-CONTEXT: JSON data, line 1: { "a":...`
	`158`	`+CONTEXT: JSON data, line 1: { "a": "\ud83d\ude04...`
`159`	`159`	`SELECT jsonb '{ "a": "\ud83d\ud83d" }' -> 'a'; -- 2 high surrogates in a row`
`160`	`160`	`ERROR: invalid input syntax for type json`
`161`	`161`	`LINE 1: SELECT jsonb '{ "a": "\ud83d\ud83d" }' -> 'a';`
`162`	`162`	`^`
`163`	`163`	`DETAIL: Unicode high surrogate must not follow a high surrogate.`
`164`		`-CONTEXT: JSON data, line 1: { "a":...`
	`164`	`+CONTEXT: JSON data, line 1: { "a": "\ud83d\ud83d...`
`165`	`165`	`SELECT jsonb '{ "a": "\ude04\ud83d" }' -> 'a'; -- surrogates in wrong order`
`166`	`166`	`ERROR: invalid input syntax for type json`
`167`	`167`	`LINE 1: SELECT jsonb '{ "a": "\ude04\ud83d" }' -> 'a';`
`168`	`168`	`^`
`169`	`169`	`DETAIL: Unicode low surrogate must follow a high surrogate.`
`170`		`-CONTEXT: JSON data, line 1: { "a":...`
	`170`	`+CONTEXT: JSON data, line 1: { "a": "\ude04...`
`171`	`171`	`SELECT jsonb '{ "a": "\ud83dX" }' -> 'a'; -- orphan high surrogate`
`172`	`172`	`ERROR: invalid input syntax for type json`
`173`	`173`	`LINE 1: SELECT jsonb '{ "a": "\ud83dX" }' -> 'a';`
`174`	`174`	`^`
`175`	`175`	`DETAIL: Unicode low surrogate must follow a high surrogate.`
`176`		`-CONTEXT: JSON data, line 1: { "a":...`
	`176`	`+CONTEXT: JSON data, line 1: { "a": "\ud83dX...`
`177`	`177`	`SELECT jsonb '{ "a": "\ude04X" }' -> 'a'; -- orphan low surrogate`
`178`	`178`	`ERROR: invalid input syntax for type json`
`179`	`179`	`LINE 1: SELECT jsonb '{ "a": "\ude04X" }' -> 'a';`
`180`	`180`	`^`
`181`	`181`	`DETAIL: Unicode low surrogate must follow a high surrogate.`
`182`		`-CONTEXT: JSON data, line 1: { "a":...`
	`182`	`+CONTEXT: JSON data, line 1: { "a": "\ude04...`
`183`	`183`	`-- handling of simple unicode escapes`
`184`	`184`	`SELECT jsonb '{ "a": "the Copyright \u00a9 sign" }' as correct_in_utf8;`
`185`	`185`	`ERROR: unsupported Unicode escape sequence`
`186`	`186`	`LINE 1: SELECT jsonb '{ "a": "the Copyright \u00a9 sign" }' as corr...`
`187`	`187`	`^`
`188`	`188`	`DETAIL: Unicode escape values cannot be used for code point values above 007F when the server encoding is not UTF8.`
`189`		`-CONTEXT: JSON data, line 1: { "a":...`
	`189`	`+CONTEXT: JSON data, line 1: { "a": "the Copyright \u00a9...`
`190`	`190`	`SELECT jsonb '{ "a": "dollar \u0024 character" }' as correct_everywhere;`
`191`	`191`	`correct_everywhere`
`192`	`192`	`-----------------------------`
`@@ -204,7 +204,7 @@ ERROR: unsupported Unicode escape sequence`
`204`	`204`	`LINE 1: SELECT jsonb '{ "a": "null \u0000 escape" }' as fails;`
`205`	`205`	`^`
`206`	`206`	`DETAIL: \u0000 cannot be converted to text.`
`207`		`-CONTEXT: JSON data, line 1: { "a":...`
	`207`	`+CONTEXT: JSON data, line 1: { "a": "null \u0000...`
`208`	`208`	`SELECT jsonb '{ "a": "null \\u0000 escape" }' as not_an_escape;`
`209`	`209`	`not_an_escape`
`210`	`210`	`------------------------------`
`@@ -216,7 +216,7 @@ ERROR: unsupported Unicode escape sequence`
`216`	`216`	`LINE 1: SELECT jsonb '{ "a": "the Copyright \u00a9 sign" }' ->> 'a'...`
`217`	`217`	`^`
`218`	`218`	`DETAIL: Unicode escape values cannot be used for code point values above 007F when the server encoding is not UTF8.`
`219`		`-CONTEXT: JSON data, line 1: { "a":...`
	`219`	`+CONTEXT: JSON data, line 1: { "a": "the Copyright \u00a9...`
`220`	`220`	`SELECT jsonb '{ "a": "dollar \u0024 character" }' ->> 'a' as correct_everywhere;`
`221`	`221`	`correct_everywhere`
`222`	`222`	`--------------------`
`@@ -234,7 +234,7 @@ ERROR: unsupported Unicode escape sequence`
`234`	`234`	`LINE 1: SELECT jsonb '{ "a": "null \u0000 escape" }' ->> 'a' as fai...`
`235`	`235`	`^`
`236`	`236`	`DETAIL: \u0000 cannot be converted to text.`
`237`		`-CONTEXT: JSON data, line 1: { "a":...`
	`237`	`+CONTEXT: JSON data, line 1: { "a": "null \u0000...`
`238`	`238`	`SELECT jsonb '{ "a": "null \\u0000 escape" }' ->> 'a' as not_an_escape;`
`239`	`239`	`not_an_escape`
`240`	`240`	`--------------------`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitc25a929

File tree

3 files changed

3 files changed

`‎src/backend/utils/adt/json.c`

`‎src/test/regress/expected/json_encoding.out`

`‎src/test/regress/expected/json_encoding_1.out`

0 commit comments