lib/libxml2
1
0
Fork 0
mirror of https://gitlab.gnome.org/GNOME/libxml2.git synced 2025-07-30 22:43:14 +03:00
Code Activity

Memory issues on the new type:

Browse Source 
- xpath.c: when copying a XSLT tree object the tree need to be copied
  too, and deallocation need to occur the same way.
Daniel
This commit is contained in:
Daniel Veillard
2001-01-22 10:45:16 +00:00
parent e456646d5d
commit 8a7642fd88
2 changed files with 41 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
ChangeLog
View File

@ -1,3 +1,8 @@
Mon Jan 22 11:43:21 CET 2001 Daniel Veillard <Daniel.Veillard@imag.fr>
* xpath.c: when copying a XSLT tree object teh tree need to be copied
too, and deallocation need to occur the same way.
Mon Jan 22 10:35:40 CET 2001 Daniel Veillard <Daniel.Veillard@imag.fr> Mon Jan 22 10:35:40 CET 2001 Daniel Veillard <Daniel.Veillard@imag.fr>
* xpathInternals.h xpath.[ch] debugXML.c: added the XPATH_XSLT_TREE * xpathInternals.h xpath.[ch] debugXML.c: added the XPATH_XSLT_TREE

37
xpath.c
View File

@ -726,6 +726,34 @@ xmlXPathFreeNodeSet(xmlNodeSetPtr obj) {
xmlFree(obj); xmlFree(obj);
} }
/**
* xmlXPathFreeValueTree:
* @obj: the xmlNodeSetPtr to free
*
* Free the NodeSet compound and the actual tree, this is different
* from xmlXPathFreeNodeSet()
*/
void
xmlXPathFreeValueTree(xmlNodeSetPtr obj) {
int i;
if (obj == NULL) return;
for (i = 0;i < obj->nodeNr;i++)
if (obj->nodeTab[i] != NULL)
xmlFreeNode(obj->nodeTab[i]);
if (obj->nodeTab != NULL) {
#ifdef DEBUG
memset(obj->nodeTab, 0xB , (size_t) sizeof(xmlNodePtr) * obj->nodeMax);
#endif
xmlFree(obj->nodeTab);
}
#ifdef DEBUG
memset(obj, 0xB , (size_t) sizeof(xmlNodeSet));
#endif
xmlFree(obj);
}
#if defined(DEBUG) || defined(DEBUG_STEP) #if defined(DEBUG) || defined(DEBUG_STEP)
/** /**
* xmlGenericErrorContextNodeSet: * xmlGenericErrorContextNodeSet:
@ -1332,6 +1360,13 @@ xmlXPathObjectCopy(xmlXPathObjectPtr val) {
case XPATH_STRING: case XPATH_STRING:
ret->stringval = xmlStrdup(val->stringval); ret->stringval = xmlStrdup(val->stringval);
case XPATH_XSLT_TREE: case XPATH_XSLT_TREE:
if ((val->nodesetval != NULL) &&
(val->nodesetval->nodeTab != NULL))
ret->nodesetval = xmlXPathNodeSetCreate(
xmlCopyNode(val->nodesetval->nodeTab[0], 1));
else
ret->nodesetval = xmlXPathNodeSetCreate(NULL);
break;
case XPATH_NODESET: case XPATH_NODESET:
ret->nodesetval = xmlXPathNodeSetMerge(NULL, val->nodesetval); ret->nodesetval = xmlXPathNodeSetMerge(NULL, val->nodesetval);
break; break;
@ -1375,7 +1410,7 @@ xmlXPathFreeObject(xmlXPathObjectPtr obj) {
xmlFree(obj->stringval); xmlFree(obj->stringval);
} else if (obj->type == XPATH_XSLT_TREE) { } else if (obj->type == XPATH_XSLT_TREE) {
if (obj->nodesetval != NULL) if (obj->nodesetval != NULL)
xmlXPathFreeNodeSet(obj->nodesetval); xmlXPathFreeValueTree(obj->nodesetval);
} }
#ifdef DEBUG #ifdef DEBUG