xml.c: Process XML Inclusions recursively.

If processing an XInclude results in new <xi:include> elements, we
need to run XInclude processing again. This continues until no
replacement occurs or an error is encountered.

There is a separate issue with dynamic strings (ast_str) that will be
addressed separately.

Resolves: #65
(cherry picked from commit 19973b73fd)
This commit is contained in:
Sean Bright 2023-05-09 09:27:33 -04:00 committed by Asterisk Development Team
parent 6812357bef
commit 0e4a9d7251
1 changed files with 27 additions and 4 deletions

View File

@ -68,6 +68,29 @@ int ast_xml_finish(void)
return 0;
}
/*!
* \internal
* \brief Process XML Inclusions (XInclude).
*
* XIncludes can result in new includes being inserted, so we need to reprocess
* until no changes are made or we encounter an error.
*
* \param doc the document to process
*
* \retval 0 if XInclude processing concluded successfully
* \retval -1 if an error occurred during XInclude processing
*/
static int process_xincludes(xmlDoc *doc)
{
int res;
do {
res = xmlXIncludeProcess(doc);
} while (res > 0);
return res;
}
struct ast_xml_doc *ast_xml_open(char *filename)
{
xmlDoc *doc;
@ -84,7 +107,7 @@ struct ast_xml_doc *ast_xml_open(char *filename)
}
/* process xinclude elements. */
if (xmlXIncludeProcess(doc) < 0) {
if (process_xincludes(doc) < 0) {
xmlFreeDoc(doc);
return NULL;
}
@ -178,7 +201,7 @@ struct ast_xml_doc *ast_xml_read_memory(char *buffer, size_t size)
if (!(doc = xmlParseMemory(buffer, (int) size))) {
/* process xinclude elements. */
if (xmlXIncludeProcess(doc) < 0) {
if (process_xincludes(doc) < 0) {
xmlFreeDoc(doc);
return NULL;
}
@ -489,7 +512,7 @@ struct ast_xslt_doc *ast_xslt_open(char *filename)
return NULL;
}
if (xmlXIncludeProcess(xml) < 0) {
if (process_xincludes(xml) < 0) {
xmlFreeDoc(xml);
return NULL;
}
@ -518,7 +541,7 @@ struct ast_xslt_doc *ast_xslt_read_memory(char *buffer, size_t size)
return NULL;
}
if (xmlXIncludeProcess(doc) < 0) {
if (process_xincludes(doc) < 0) {
xmlFreeDoc(doc);
return NULL;
}