Hi all,
The following patch is small, but given the current point in the cycle I
fully understand that it may be held for an 8.0.1 or 8.1. I'm sorry that I
didn't stumble across it sooner...
It fixes a segfault due to the XML parser not being cleared up as required
when the xpath_list function is applied to an invalid document. The change
just ensures that the early exit (on NULL input) from the
pgxml_result_to_text function calls xmlCleanupParser, as the main exit
does.
Regards
John Gray
Index: contrib/xml2/xpath.c
===================================================================
RCS file: /projects/cvsroot/pgsql/contrib/xml2/xpath.c,v
retrieving revision 1.5
diff -c -r1.5 xpath.c
*** contrib/xml2/xpath.c 2 Dec 2004 22:21:12 -0000 1.5
--- contrib/xml2/xpath.c 9 Jan 2005 17:17:06 -0000
***************
*** 582,588 ****
--- 582,591 ----
text *xpres;
if (res == NULL)
+ {
+ xmlCleanupParser();
return NULL;
+ }
switch (res->type)
{
case XPATH_NODESET:
---------------------------(end of broadcast)---------------------------
TIP 4: Don't 'kill -9' the postmaster
