From: Tom Lane <tgl@sss.pgh.pa.us>
Date: Thu, 3 Jul 2008 00:04:24 +0000 (+0000)
Subject: Fix transaction-lifespan memory leak in xpath().  Report by Matt Magoffin,
X-Git-Tag: REL8_4_BETA1~1211
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=c5f4b98faeeb8b41b2fd8335b91bb94b7cff3a25;p=postgresql

Fix transaction-lifespan memory leak in xpath().  Report by Matt Magoffin,
fix by Kris Jurka.
---

diff --git a/src/backend/utils/adt/xml.c b/src/backend/utils/adt/xml.c
index 514e5ae2bc..799e9a283a 100644
--- a/src/backend/utils/adt/xml.c
+++ b/src/backend/utils/adt/xml.c
@@ -7,7 +7,7 @@
  * Portions Copyright (c) 1996-2008, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/backend/utils/adt/xml.c,v 1.74 2008/05/12 00:00:51 alvherre Exp $
+ * $PostgreSQL: pgsql/src/backend/utils/adt/xml.c,v 1.75 2008/07/03 00:04:24 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -3160,6 +3160,7 @@ xml_xmlnodetoxmltype(xmlNodePtr cur)
 	{
 		str = xmlXPathCastNodeToString(cur);
 		result = (xmltype *) cstring_to_text((char *) str);
+		xmlFree(str);
 	}
 
 	return result;