diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2010-03-01 05:17:01 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2010-03-01 05:17:01 +0000 |
| commit | a8ab473383e3b085cee3fabf705130c476ce95c9 (patch) | |
| tree | ab66b219817dced9269b67d5970514c0cb859a67 /contrib/xml2/xslt_proc.c | |
| parent | 94152f93f562ad040d40a8f53cc5917dc135f584 (diff) | |
Remove xmlCleanupParser calls from contrib/xml2.
These are unnecessary and probably dangerous. I don't see any immediate
risk situations in the core XML support or contrib/xml2 itself, but there
could be issues with external uses of libxml2, and in any case it's an
accident waiting to happen.
Diffstat (limited to 'contrib/xml2/xslt_proc.c')
| -rw-r--r-- | contrib/xml2/xslt_proc.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/contrib/xml2/xslt_proc.c b/contrib/xml2/xslt_proc.c index 51461695d81..7fb3b294ee0 100644 --- a/contrib/xml2/xslt_proc.c +++ b/contrib/xml2/xslt_proc.c @@ -80,7 +80,6 @@ xslt_process(PG_FUNCTION_ARGS) if (doctree == NULL) { - xmlCleanupParser(); elog_error("Error parsing XML document", false); PG_RETURN_NULL(); @@ -94,7 +93,6 @@ xslt_process(PG_FUNCTION_ARGS) if (ssdoc == NULL) { xmlFreeDoc(doctree); - xmlCleanupParser(); elog_error("Error parsing stylesheet as XML document", false); PG_RETURN_NULL(); } @@ -109,7 +107,6 @@ xslt_process(PG_FUNCTION_ARGS) { xmlFreeDoc(doctree); xsltCleanupGlobals(); - xmlCleanupParser(); elog_error("Failed to parse stylesheet", false); PG_RETURN_NULL(); } @@ -122,7 +119,6 @@ xslt_process(PG_FUNCTION_ARGS) xmlFreeDoc(doctree); xsltCleanupGlobals(); - xmlCleanupParser(); if (resstat < 0) PG_RETURN_NULL(); |