NotificationsYou must be signed in to change notification settings
Fork5.2k
Star19k

Commitd5f014d

committed

Remove unnecessary complication around xmlParseBalancedChunkMemory.

When I prepared71c0921 et al yesterday, I was thinking that thelogic involving explicitly freeing the node_list output was stillneeded to dodge leakage bugs in libxml2. But I was misremembering:we introduced that only because with early 2.13.x releases we couldnot trust xmlParseBalancedChunkMemory's result code, so we had tolook to see if a node list was returned or not. There's no reasonto believe that xmlParseBalancedChunkMemory will fail to clean upthe node list when required, so simplify. (This essentiallycompletes reverting all the non-cosmetic changes in6082b3d.)Reported-by: Jim Jones <jim.jones@uni-muenster.de>Author: Tom Lane <tgl@sss.pgh.pa.us>Discussion:https://postgr.es/m/997668.1753802857@sss.pgh.pa.usBackpatch-through: 13

1 parent1fe9e38 commitd5f014dCopy full SHA for d5f014d

File tree

1 file changed

-9

lines changed

src/backend/utils/adt
- xml.c

1 file changed

-9

lines changed

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

Lines changed: 1 addition & 9 deletions

Original file line number	Diff line number	Diff line change
`@@ -1883,24 +1883,16 @@ xml_parse(text *data, XmlOptionType xmloption_arg,`
`1883`	`1883`	`/* allow empty content */`
`1884`	`1884`	`if (*(utf8string+count))`
`1885`	`1885`	`{`
`1886`		`-xmlNodePtrnode_list=NULL;`
`1887`		`-`
`1888`	`1886`	`res_code=xmlParseBalancedChunkMemory(doc,NULL,NULL,0,`
`1889`	`1887`	`utf8string+count,`
`1890`		`-&node_list);`
	`1888`	`+parsed_nodes);`
`1891`	`1889`	`if (res_code!=0\|\|xmlerrcxt->err_occurred)`
`1892`	`1890`	`{`
`1893`	`1891`	`xml_errsave(escontext,xmlerrcxt,`
`1894`	`1892`	`ERRCODE_INVALID_XML_CONTENT,`
`1895`	`1893`	`"invalid XML content");`
`1896`		`-xmlFreeNodeList(node_list);`
`1897`	`1894`	`gotofail;`
`1898`	`1895`	`}`
`1899`		`-`
`1900`		`-if (parsed_nodes!=NULL)`
`1901`		`-*parsed_nodes=node_list;`
`1902`		`-else`
`1903`		`-xmlFreeNodeList(node_list);`
`1904`	`1896`	`}`
`1905`	`1897`	`}`
`1906`	`1898`

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!

Commitd5f014d

File tree

1 file changed

1 file changed

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

0 commit comments