details: https://github.com/nginx/njs/commit/597b1fd1fa34999ec47bbd574da35491f56b8893 branches: master commit: 597b1fd1fa34999ec47bbd574da35491f56b8893 user: Dmitry Volyntsev <xei...@nginx.com> date: Wed, 6 Nov 2024 22:08:21 -0800 description: XML: fixed tests with libxml2 2.13 and later.
This fixes #812 issue on Github. --- external/njs_xml_module.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/external/njs_xml_module.c b/external/njs_xml_module.c index 86c896c0..d9f3bbb5 100644 --- a/external/njs_xml_module.c +++ b/external/njs_xml_module.c @@ -1275,17 +1275,13 @@ njs_xml_node_tags_handler(njs_vm_t *vm, xmlNode *current, njs_str_t *name, /* set or delete. */ - copy = xmlDocCopyNode(current, current->doc, 1); + copy = xmlDocCopyNode(current, current->doc, + 2 /* copy properties and namespaces */); if (njs_slow_path(copy == NULL)) { njs_vm_internal_error(vm, "xmlDocCopyNode() failed"); return NJS_ERROR; } - if (copy->children != NULL) { - xmlFreeNodeList(copy->children); - copy->children = NULL; - } - if (retval == NULL) { /* delete. */ return njs_xml_replace_node(vm, current, copy); _______________________________________________ nginx-devel mailing list nginx-devel@nginx.org https://mailman.nginx.org/mailman/listinfo/nginx-devel