From: Tom Lane <tgl@sss.pgh.pa.us>
Date: Thu, 3 Jul 2008 00:04:34 +0000 (+0000)
Subject: Fix transaction-lifespan memory leak in xpath().  Report by Matt Magoffin,
X-Git-Tag: REL8_3_4~43
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=4dad6c94fe79cc1edcf6957bed7471685a519fe8;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 21341a03a7..7a627b7683 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.68.2.2 2008/03/24 19:12:58 tgl Exp $
+ * $PostgreSQL: pgsql/src/backend/utils/adt/xml.c,v 1.68.2.3 2008/07/03 00:04:34 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -3201,6 +3201,7 @@ xml_xmlnodetoxmltype(xmlNodePtr cur)
 		result = (text *) palloc(len + VARHDRSZ);
 		SET_VARSIZE(result, len + VARHDRSZ);
 		memcpy(VARDATA(result), str, len);
+		xmlFree(str);
 	}
 
 	return result;