external/libxmlsec/xmlsec1-ooxml.patch.1 | 84 ++++++++++++++++++++++++++-----
1 file changed, 73 insertions(+), 11 deletions(-)
New commits:
commit b2734fcc25cc30bb297b50ee5a7f37cfb5667050
Author: Miklos Vajna <vmik...@collabora.co.uk>
Date: Wed Jan 27 16:02:49 2016 +0100
libxmlsec: sort elements in OOXML RelationshipTransform
The spec says that the implementer shall sort relationship elements by
Id value in lexicographical order, so do that before the filtering of
these elements.
With this, all streams validate for a test document that is supposed to
be valid, though xmlSecDSigCtxVerify() still reports errors.
Change-Id: I9d9cd511eaebad1f13f4e06891b2a3f61fee4500
diff --git a/external/libxmlsec/xmlsec1-ooxml.patch.1
b/external/libxmlsec/xmlsec1-ooxml.patch.1
index a673416..a0878a5 100644
--- a/external/libxmlsec/xmlsec1-ooxml.patch.1
+++ b/external/libxmlsec/xmlsec1-ooxml.patch.1
@@ -1,4 +1,4 @@
-From 7e46355a9a7a223077f4d83587fd842bbaf97e37 Mon Sep 17 00:00:00 2001
+From 24c1395c547f8a2f0c78a8618c766ecd55909c66 Mon Sep 17 00:00:00 2001
From: Miklos Vajna <vmik...@collabora.co.uk>
Date: Mon, 25 Jan 2016 09:50:03 +0100
Subject: [PATCH] OOXML Relationship Transform
@@ -8,8 +8,8 @@ Subject: [PATCH] OOXML Relationship Transform
include/xmlsec/transforms.h | 4 +
src/strings.c | 3 +
src/transforms.c | 11 +
- src/xpath.c | 480 ++++++++++++++++++++++++++++++++++++++++++++
- 5 files changed, 501 insertions(+)
+ src/xpath.c | 542 ++++++++++++++++++++++++++++++++++++++++++++
+ 5 files changed, 563 insertions(+)
diff --git a/include/xmlsec/strings.h b/include/xmlsec/strings.h
index 07afb9d..9c72d1b 100644
@@ -77,7 +77,7 @@ index 2ed3fe8..9e5ad27 100644
if(xmlSecTransformIdsRegister(xmlSecTransformXsltId) < 0) {
xmlSecError(XMLSEC_ERRORS_HERE,
diff --git a/src/xpath.c b/src/xpath.c
-index 8b0b4f8..e56920e 100644
+index 8b0b4f8..b9a03e5 100644
--- a/src/xpath.c
+++ b/src/xpath.c
@@ -17,6 +17,7 @@
@@ -88,7 +88,7 @@ index 8b0b4f8..e56920e 100644
#include <xmlsec/xmlsec.h>
#include <xmlsec/xmltree.h>
-@@ -1144,5 +1145,484 @@ xmlSecTransformVisa3DHackExecute(xmlSecTransformPtr
transform, int last,
+@@ -1144,5 +1145,546 @@ xmlSecTransformVisa3DHackExecute(xmlSecTransformPtr
transform, int last,
return(0);
}
@@ -271,10 +271,10 @@ index 8b0b4f8..e56920e 100644
+ xmlOutputBufferWriteString(buf, (const char *)xmlGetProp(cur,
attr->name));
+ xmlOutputBufferWriteString(buf, "\"");
+ }
-
++
+ if (strcmp(cur->name, "Relationship") == 0 && !foundTargetMode)
+ xmlOutputBufferWriteString(buf, " TargetMode=\"Internal\"");
-
++
+ xmlOutputBufferWriteString(buf, ">");
+
+ if (cur->children)
@@ -331,17 +331,79 @@ index 8b0b4f8..e56920e 100644
+ return 0;
+}
+
++/* Sorts Relationship elements by Id value in lexicographical order. */
++static int xmlSecTransformRelationshipCompare(xmlNodePtr node1, xmlNodePtr
node2)
++{
++ xmlChar* id1;
++ xmlChar* id2;
++
++ if (node1 == node2)
++ return 0;
++ if (!node1)
++ return -1;
++ if (!node2)
++ return 1;
++
++ id1 = xmlGetProp(node1, "Id");
++ id2 = xmlGetProp(node2, "Id");
++
++ if (!id1)
++ return -1;
++ if (!id2)
++ return 1;
++
++ return xmlStrcmp(id1, id2);
++}
++
+int xmlSecTransformRelationshipProcessNodeList(xmlSecTransformPtr transform,
xmlOutputBufferPtr buf, xmlNodePtr cur)
+{
++ xmlListPtr list;
+ int ret;
+
-+ for (ret = 0; cur; cur = cur->next)
++ list = xmlListCreate(NULL,
(xmlListDataCompare)xmlSecTransformRelationshipCompare);
++ if (!list)
++ return -1;
++
++ for (; cur; cur = cur->next)
+ {
-+ ret = xmlSecTransformRelationshipProcessNode(transform, buf, cur);
-+ if (ret < 0)
-+ return -1;
++ if (strcmp(cur->name, "Relationship") == 0)
++ xmlListInsert(list, cur);
++ else
++ {
++ ret = xmlSecTransformRelationshipProcessNode(transform, buf, cur);
++ if (ret < 0)
++ {
++ xmlListDelete(list);
++ return -1;
++ }
++ }
++ }
++
++ if (!xmlListEmpty(list))
++ {
++ int size;
++ int i;
+
++ xmlListSort(list);
++ size = xmlListSize(list);
++ for (i = 0; i < size; ++i)
++ {
++ xmlLinkPtr link = xmlListFront(list);
++ xmlNodePtr node = (xmlNodePtr)xmlLinkGetData(link);
+
++ ret = xmlSecTransformRelationshipProcessNode(transform, buf,
node);
++ if (ret < 0)
++ {
++ xmlListDelete(list);
++ return -1;
++ }
++
++ xmlListPopFront(list);
++ }
++
+ }
+
++ xmlListDelete(list);
+ return 0;
+}
+
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
