From 1f7e7d8cadf5c45585a596a8e2243f074484acb8 Mon Sep 17 00:00:00 2001 From: liningjie Date: Fri, 1 Sep 2023 18:28:47 +0800 Subject: [PATCH] parser: Fix old SAX1 parser with custom callbacks --- ...ays-initialize-SAX1-element-handlers.patch | 67 +++++++++++++++++++ ...ld-SAX1-parser-with-custom-callbacks.patch | 32 +++++++++ libxml2.spec | 7 +- 3 files changed, 105 insertions(+), 1 deletion(-) create mode 100644 backport-Always-initialize-SAX1-element-handlers.patch create mode 100644 backport-Fix-old-SAX1-parser-with-custom-callbacks.patch diff --git a/backport-Always-initialize-SAX1-element-handlers.patch b/backport-Always-initialize-SAX1-element-handlers.patch new file mode 100644 index 0000000..ec99681 --- /dev/null +++ b/backport-Always-initialize-SAX1-element-handlers.patch @@ -0,0 +1,67 @@ +From 235b15a590eecf97b09e87bdb7e4f8333e9de129 Mon Sep 17 00:00:00 2001 +From: Nick Wellnhofer +Date: Mon, 8 May 2023 17:58:02 +0200 +Subject: [PATCH] SAX: Always initialize SAX1 element handlers + +Follow-up to commit d0c3f01e. A parser context will be initialized to +SAX version 2, but this can be overridden with XML_PARSE_SAX1 later, +so we must initialize the SAX1 element handlers as well. + +Change the check in xmlDetectSAX2 to only look for XML_SAX2_MAGIC, so +we don't switch to SAX1 if the SAX2 element handlers are NULL. +--- + SAX2.c | 11 +++++++---- + parser.c | 5 +---- + 2 files changed, 8 insertions(+), 8 deletions(-) + +diff --git a/SAX2.c b/SAX2.c +index b6be1a6c4..910193892 100644 +--- a/SAX2.c ++++ b/SAX2.c +@@ -2874,20 +2874,23 @@ xmlSAXVersion(xmlSAXHandler *hdlr, int version) + { + if (hdlr == NULL) return(-1); + if (version == 2) { +- hdlr->startElement = NULL; +- hdlr->endElement = NULL; + hdlr->startElementNs = xmlSAX2StartElementNs; + hdlr->endElementNs = xmlSAX2EndElementNs; + hdlr->serror = NULL; + hdlr->initialized = XML_SAX2_MAGIC; + #ifdef LIBXML_SAX1_ENABLED + } else if (version == 1) { +- hdlr->startElement = xmlSAX2StartElement; +- hdlr->endElement = xmlSAX2EndElement; + hdlr->initialized = 1; + #endif /* LIBXML_SAX1_ENABLED */ + } else + return(-1); ++#ifdef LIBXML_SAX1_ENABLED ++ hdlr->startElement = xmlSAX2StartElement; ++ hdlr->endElement = xmlSAX2EndElement; ++#else ++ hdlr->startElement = NULL; ++ hdlr->endElement = NULL; ++#endif /* LIBXML_SAX1_ENABLED */ + hdlr->internalSubset = xmlSAX2InternalSubset; + hdlr->externalSubset = xmlSAX2ExternalSubset; + hdlr->isStandalone = xmlSAX2IsStandalone; +diff --git a/parser.c b/parser.c +index 0c8bed129..e133fe0a6 100644 +--- a/parser.c ++++ b/parser.c +@@ -842,10 +842,7 @@ xmlDetectSAX2(xmlParserCtxtPtr ctxt) { + if (ctxt == NULL) return; + sax = ctxt->sax; + #ifdef LIBXML_SAX1_ENABLED +- if ((sax) && (sax->initialized == XML_SAX2_MAGIC) && +- ((sax->startElementNs != NULL) || +- (sax->endElementNs != NULL) || +- ((sax->startElement == NULL) && (sax->endElement == NULL)))) ++ if ((sax) && (sax->initialized == XML_SAX2_MAGIC)) + ctxt->sax2 = 1; + #else + ctxt->sax2 = 1; +-- +GitLab + diff --git a/backport-Fix-old-SAX1-parser-with-custom-callbacks.patch b/backport-Fix-old-SAX1-parser-with-custom-callbacks.patch new file mode 100644 index 0000000..50fba55 --- /dev/null +++ b/backport-Fix-old-SAX1-parser-with-custom-callbacks.patch @@ -0,0 +1,32 @@ +From d0c3f01e110d54415611c5fa0040cdf4a56053f9 Mon Sep 17 00:00:00 2001 +From: Nick Wellnhofer +Date: Sat, 6 May 2023 17:47:37 +0200 +Subject: [PATCH] parser: Fix old SAX1 parser with custom callbacks + +For some reason, xmlCtxtUseOptionsInternal set the start and end element +SAX handlers to the internal DOM builder functions when XML_PARSE_SAX1 +was specified. This means that custom SAX handlers could never work with +that flag because these functions would receive the wrong user data +argument and crash immediately. + +Fixes #535. +--- + parser.c | 2 -- + 1 file changed, 2 deletions(-) + +diff --git a/parser.c b/parser.c +index bb05791d3..0c8bed129 100644 +--- a/parser.c ++++ b/parser.c +@@ -14479,8 +14479,6 @@ xmlCtxtUseOptionsInternal(xmlParserCtxtPtr ctxt, int options, const char *encodi + } + #ifdef LIBXML_SAX1_ENABLED + if (options & XML_PARSE_SAX1) { +- ctxt->sax->startElement = xmlSAX2StartElement; +- ctxt->sax->endElement = xmlSAX2EndElement; + ctxt->sax->startElementNs = NULL; + ctxt->sax->endElementNs = NULL; + ctxt->sax->initialized = 1; +-- +GitLab + diff --git a/libxml2.spec b/libxml2.spec index 6a221e1..cf66d95 100644 --- a/libxml2.spec +++ b/libxml2.spec @@ -1,12 +1,14 @@ Summary: Library providing XML and HTML support Name: libxml2 Version: 2.11.4 -Release: 3 +Release: 4 License: MIT Group: Development/Libraries Source: https://download.gnome.org/sources/%{name}/2.11/%{name}-%{version}.tar.xz Patch0: libxml2-multilib.patch +Patch1: backport-Fix-old-SAX1-parser-with-custom-callbacks.patch +Patch2: backport-Always-initialize-SAX1-element-handlers.patch BuildRoot: %{_tmppath}/%{name}-%{version}-root BuildRequires: python3-devel @@ -158,6 +160,9 @@ rm -fr %{buildroot} %changelog +* Fri Sep 01 2023 liningjie - 2.11.4-4 +- Fix old SAX1 parser with custom callbacks + * Mon Aug 07 2023 zhuofeng - 2.11.4-3 - Type:bugfix - CVE:NA -- Gitee