96 lines
3.1 KiB
Diff
96 lines
3.1 KiB
Diff
From 345d6826d0eae6f0a962456b8ed6f6a1bad0877d Mon Sep 17 00:00:00 2001
|
|
From: David Kilzer <ddkilzer@apple.com>
|
|
Date: Sat, 24 May 2025 15:06:42 -0700
|
|
Subject: [PATCH] libxslt: Type confusion in xmlNode.psvi between stylesheet
|
|
and source nodes
|
|
|
|
* libxslt/functions.c:
|
|
(xsltDocumentFunctionLoadDocument):
|
|
- Implement fix suggested by Ivan Fratric. This copies the xmlDoc,
|
|
calls xsltCleanupSourceDoc() to remove pvsi fields, then adds the
|
|
xmlDoc to tctxt->docList.
|
|
- Add error handling for functions that may return NULL.
|
|
* libxslt/transform.c:
|
|
- Remove static keyword so this can be called from
|
|
xsltDocumentFunctionLoadDocument().
|
|
* libxslt/transformInternals.h: Add.
|
|
(xsltCleanupSourceDoc): Add declaration.
|
|
|
|
Fixes #139.
|
|
---
|
|
libxslt/functions.c | 16 +++++++++++++++-
|
|
libxslt/transform.c | 3 ++-
|
|
libxslt/transformInternals.h | 9 +++++++++
|
|
3 files changed, 26 insertions(+), 2 deletions(-)
|
|
create mode 100644 libxslt/transformInternals.h
|
|
|
|
Index: libxslt-1.1.38/libxslt/functions.c
|
|
===================================================================
|
|
--- libxslt-1.1.38.orig/libxslt/functions.c
|
|
+++ libxslt-1.1.38/libxslt/functions.c
|
|
@@ -34,6 +34,7 @@
|
|
#include "numbersInternals.h"
|
|
#include "keys.h"
|
|
#include "documents.h"
|
|
+#include "transformInternals.h"
|
|
|
|
#ifdef WITH_XSLT_DEBUG
|
|
#define WITH_XSLT_DEBUG_FUNCTION
|
|
@@ -145,7 +146,20 @@ xsltDocumentFunctionLoadDocument(xmlXPat
|
|
/*
|
|
* This selects the stylesheet's doc itself.
|
|
*/
|
|
- doc = tctxt->style->doc;
|
|
+ doc = xmlCopyDoc(tctxt->style->doc, 1);
|
|
+ if (doc == NULL) {
|
|
+ xsltTransformError(tctxt, NULL, NULL,
|
|
+ "document() : failed to copy style doc\n");
|
|
+ goto out_fragment;
|
|
+ }
|
|
+ xsltCleanupSourceDoc(doc); /* Remove psvi fields. */
|
|
+ idoc = xsltNewDocument(tctxt, doc);
|
|
+ if (idoc == NULL) {
|
|
+ xsltTransformError(tctxt, NULL, NULL,
|
|
+ "document() : failed to create xsltDocument\n");
|
|
+ xmlFreeDoc(doc);
|
|
+ goto out_fragment;
|
|
+ }
|
|
} else {
|
|
valuePush(ctxt, xmlXPathNewNodeSet(NULL));
|
|
|
|
Index: libxslt-1.1.38/libxslt/transform.c
|
|
===================================================================
|
|
--- libxslt-1.1.38.orig/libxslt/transform.c
|
|
+++ libxslt-1.1.38/libxslt/transform.c
|
|
@@ -43,6 +43,7 @@
|
|
#include "xsltlocale.h"
|
|
#include "pattern.h"
|
|
#include "transform.h"
|
|
+#include "transformInternals.h"
|
|
#include "variables.h"
|
|
#include "numbersInternals.h"
|
|
#include "namespaces.h"
|
|
@@ -5743,7 +5744,7 @@ xsltCountKeys(xsltTransformContextPtr ct
|
|
*
|
|
* Resets source node flags and ids stored in 'psvi' member.
|
|
*/
|
|
-static void
|
|
+void
|
|
xsltCleanupSourceDoc(xmlDocPtr doc) {
|
|
xmlNodePtr cur = (xmlNodePtr) doc;
|
|
void **psviPtr;
|
|
Index: libxslt-1.1.38/libxslt/transformInternals.h
|
|
===================================================================
|
|
--- /dev/null
|
|
+++ libxslt-1.1.38/libxslt/transformInternals.h
|
|
@@ -0,0 +1,9 @@
|
|
+/*
|
|
+ * Summary: set of internal interfaces for the XSLT engine transformation part.
|
|
+ *
|
|
+ * Copy: See Copyright for the status of this software.
|
|
+ *
|
|
+ * Author: David Kilzer <ddkilzer@apple.com>
|
|
+ */
|
|
+
|
|
+void xsltCleanupSourceDoc(xmlDocPtr doc);
|