From 7a14e6358a1e24505c8337759842e126ea5a20ff Mon Sep 17 00:00:00 2001
From: Christian Stocker <chregu@php.net>
Date: Tue, 8 Apr 2003 19:30:27 +0000
Subject: [PATCH] fixes memleaks in element->get_elements_by_tagname &
 document->get_elements_by_tagname

---
 ext/domxml/php_domxml.c | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/ext/domxml/php_domxml.c b/ext/domxml/php_domxml.c
index 529c1345d7..41c25746f8 100644
--- a/ext/domxml/php_domxml.c
+++ b/ext/domxml/php_domxml.c
@@ -3079,6 +3079,7 @@ PHP_FUNCTION(domxml_doc_get_elements_by_tagname)
 	xmlXPathObjectPtr xpathobjp;
 	xmlNode *contextnodep;
 	int name_len;
+	int free_context = 0;
 	char *str,*name;
 
 	contextnode = NULL;
@@ -3089,6 +3090,7 @@ PHP_FUNCTION(domxml_doc_get_elements_by_tagname)
 	/* if no xpath_context was submitted, create a new one */
 	if (ctxpin == NULL) {
 		ctxp = xmlXPathNewContext(docp);
+		free_context = 1;
 	} else {
 		DOMXML_GET_OBJ(ctxp, ctxpin, le_xpathctxp);
 	}
@@ -3119,6 +3121,10 @@ PHP_FUNCTION(domxml_doc_get_elements_by_tagname)
 
 			if (NULL == (nodesetp = xpathobjp->nodesetval)) {
 				zval_dtor(rv);
+				xmlXPathFreeObject (xpathobjp);
+				if (free_context) {
+					xmlXPathFreeContext(ctxp);
+				}
 				RETURN_FALSE;
 			}
 
@@ -3138,6 +3144,10 @@ PHP_FUNCTION(domxml_doc_get_elements_by_tagname)
 			break;
 	}
 
+	xmlXPathFreeObject(xpathobjp);
+	if (free_context) {
+		xmlXPathFreeContext(ctxp);
+	}
 	*return_value = *rv;
 	FREE_ZVAL(rv);
 }
@@ -3215,6 +3225,7 @@ PHP_FUNCTION(domxml_elem_get_elements_by_tagname)
 			zend_hash_next_index_insert(Z_ARRVAL_P(rv), &child, sizeof(zval *), NULL);
 		}
 	}
+	xmlXPathFreeNodeSet(nodesetp);
 	*return_value = *rv;
 	FREE_ZVAL(rv);
 
-- 
2.40.0