NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitd99d0e6

committed

Remove ill-conceived ban on zero length json object keys.

We removed a similar ban on this in json_object recently, but the ban indatum_to_json was left, which generate4d sprutious errors in othee jsongenerators, notable json_build_object.Along the way, add an assertion that datum_to_json is not passed a nullkey. All current callers comply with this rule, but the assertion willcatch any possible future misbehaviour.

1 parent2024ead commitd99d0e6Copy full SHA for d99d0e6

File tree

1 file changed

-4

lines changed

src/backend/utils/adt
- json.c

1 file changed

-4

lines changed

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

Lines changed: 3 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -1341,6 +1341,9 @@ datum_to_json(Datum val, bool is_null, StringInfo result,`
`1341`	`1341`	`boolnumeric_error;`
`1342`	`1342`	`JsonLexContextdummy_lex;`
`1343`	`1343`
	`1344`	`+/* callers are expected to ensure that null keys are not passed in */`
	`1345`	`+Assert( ! (key_scalar&&is_null));`
	`1346`	`+`
`1344`	`1347`	`if (is_null)`
`1345`	`1348`	`{`
`1346`	`1349`	`appendStringInfoString(result,"null");`
`@@ -1486,10 +1489,6 @@ datum_to_json(Datum val, bool is_null, StringInfo result,`
`1486`	`1489`	`break;`
`1487`	`1490`	`default:`
`1488`	`1491`	`outputstr=OidOutputFunctionCall(outfuncoid,val);`
`1489`		`-if (key_scalar&&*outputstr=='\0')`
`1490`		`-ereport(ERROR,`
`1491`		`-(errcode(ERRCODE_INVALID_PARAMETER_VALUE),`
`1492`		`-errmsg("key value must not be empty")));`
`1493`	`1492`	`escape_json(result,outputstr);`
`1494`	`1493`	`pfree(outputstr);`
`1495`	`1494`	`break;`

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!

Commitd99d0e6

File tree

1 file changed

1 file changed

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

0 commit comments