c/libxml2
1
0
Fork 0
mirror of https://gitlab.gnome.org/GNOME/libxml2 synced 2025-03-28 21:33:13 +00:00
Code Issues Packages Projects Releases Wiki Activity

xpath: Make recursion check work with xmlXPathCompile

Browse Source 
The check for maximum recursion depth required a parser context with an
xmlXPathContext which xmlXPathCompile didn't provide.

All other code should already set up or require an xmlXPathContext.
This commit is contained in:
Nick Wellnhofer 2024-09-13 20:59:47 +02:00
parent d4e4f6f2e0
commit 7c4fef2412
1 changed files with 22 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
xpath.c
View File

@ -13030,6 +13030,7 @@ xmlXPathOptimizeExpression(xmlXPathParserContextPtr pctxt,
xmlXPathCompExprPtr
xmlXPathCtxtCompile(xmlXPathContextPtr ctxt, const xmlChar *str) {
xmlXPathParserContextPtr pctxt;
xmlXPathContextPtr tmpctxt = NULL;
xmlXPathCompExprPtr comp;
int oldDepth = 0;
@ -13041,18 +13042,32 @@ xmlXPathCtxtCompile(xmlXPathContextPtr ctxt, const xmlChar *str) {
xmlInitParser();
/*
* We need an xmlXPathContext for the depth check.
*/
if (ctxt == NULL) {
tmpctxt = xmlXPathNewContext(NULL);
if (tmpctxt == NULL)
return(NULL);
ctxt = tmpctxt;
}
pctxt = xmlXPathNewParserContext(str, ctxt);
if (pctxt == NULL)
if (pctxt == NULL) {
if (tmpctxt != NULL)
xmlXPathFreeContext(tmpctxt);
return NULL;
if (ctxt != NULL)
oldDepth = ctxt->depth;
}
oldDepth = ctxt->depth;
xmlXPathCompileExpr(pctxt, 1);
if (ctxt != NULL)
ctxt->depth = oldDepth;
ctxt->depth = oldDepth;
if( pctxt->error != XPATH_EXPRESSION_OK )
{
xmlXPathFreeParserContext(pctxt);
if (tmpctxt != NULL)
xmlXPathFreeContext(tmpctxt);
return(NULL);
}
@ -13077,6 +13092,8 @@ xmlXPathCtxtCompile(xmlXPathContextPtr ctxt, const xmlChar *str) {
pctxt->comp = NULL;
}
xmlXPathFreeParserContext(pctxt);
if (tmpctxt != NULL)
xmlXPathFreeContext(tmpctxt);
if (comp != NULL) {
comp->expr = xmlStrdup(str);