Commit06a66d8

committed

Clean up a bug in sql/json items commit66ea94e

Remove a buggy and unnecessary test, along with an unnecessary pstrdup()and a line of dead code.Per report, diagnosis and fix from Tom LaneDiscussion:https://postgr.es/m/439811.1706211069@sss.pgh.pa.us

1 parent7014c9a commit06a66d8Copy full SHA for 06a66d8

File tree

-3

lines changed

-3

lines changed

Lines changed: 1 addition & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -1540,11 +1540,9 @@ executeItemOptUnwrapTarget(JsonPathExecContext cxt, JsonPathItem jsp,`
`1540`	`1540`	`break;`
`1541`	`1541`	`}`
`1542`	`1542`
`1543`		`-res=jperOk;`
`1544`		`-`
`1545`	`1543`	`jb=&jbv;`
`1546`	`1544`	`Assert(tmp!=NULL);/* We must have set tmp above */`
`1547`		`-jb->val.string.val=(jb->type==jbvString) ?tmp :pstrdup(tmp);`
	`1545`	`+jb->val.string.val=tmp;`
`1548`	`1546`	`jb->val.string.len=strlen(jb->val.string.val);`
`1549`	`1547`	`jb->type=jbvString;`
`1550`	`1548`

Comments

(0)