diff options
| author | Mark Shroyer <code@markshroyer.com> | 2013-09-26 02:01:21 -0400 |
|---|---|---|
| committer | The Qt Project <gerrit-noreply@qt-project.org> | 2013-10-02 21:53:46 +0200 |
| commit | 3c63cedb5d3defd831eb2d8bfdeb883121737ef3 (patch) | |
| tree | 84e2ff50c2ad26ca7ed188484cb21cb2f666cf80 /tests | |
| parent | 11b150d935ebdd422e3d3edb698f2e08c45e2229 (diff) | |
| download | qtxmlpatterns-3c63cedb5d3defd831eb2d8bfdeb883121737ef3.tar.gz | |
Fix undefined behavior validating XSD substitution groups
A bug in XSD substitution group validation would result in an invalid
cast of SchemaType::Ptr to XsdComplexType::Ptr, in which case evaluating
complexType->prohibitedSubstitutions() exhibited undefined behavior. In
practice this caused validation against XSD schemas containing
substitution groups to fail on some machines, where ORing the checkSet
mask against out of bounds memory could cause the function
XsdSchemaHelper::substitutionGroupOkTransitive()
to return a false negative.
Minus the bug fix, the regression test added in this commit failed on
(at least) Linux ARM when compiled with the Linaro toolchain 2013.01 g++
4.7, with flags -marm -mcpu=cortex-a8 -O2. However, it did not fail on
a Linux amd64 machine prior to the bug fix.
Change-Id: Idd060a941a3bc0620f1fcc903375e43022bdcbdc
Reviewed-by: Lars Knoll <lars.knoll@digia.com>
Diffstat (limited to 'tests')
4 files changed, 33 insertions, 0 deletions
diff --git a/tests/auto/xmlpatternsvalidator/files/substitution-group-invalid.xml b/tests/auto/xmlpatternsvalidator/files/substitution-group-invalid.xml new file mode 100644 index 0000000..a4ca0c6 --- /dev/null +++ b/tests/auto/xmlpatternsvalidator/files/substitution-group-invalid.xml @@ -0,0 +1,3 @@ +<outer> + <innerInvalid>foo</innerInvalid> +</outer> diff --git a/tests/auto/xmlpatternsvalidator/files/substitution-group-valid.xml b/tests/auto/xmlpatternsvalidator/files/substitution-group-valid.xml new file mode 100644 index 0000000..54d81a0 --- /dev/null +++ b/tests/auto/xmlpatternsvalidator/files/substitution-group-valid.xml @@ -0,0 +1,3 @@ +<outer> + <innerValid>foo</innerValid> +</outer> diff --git a/tests/auto/xmlpatternsvalidator/files/substitution-group.xsd b/tests/auto/xmlpatternsvalidator/files/substitution-group.xsd new file mode 100644 index 0000000..7143bb5 --- /dev/null +++ b/tests/auto/xmlpatternsvalidator/files/substitution-group.xsd @@ -0,0 +1,15 @@ +<?xml version="1.0" encoding="UTF-8"?> +<xs:schema xmlns:xs="http://www.w3.org/2001/XMLSchema" elementFormDefault="qualified"> + <xs:element name="outer"> + <xs:complexType> + <xs:choice> + <xs:element ref="MySub"/> + <xs:element ref="MyBadSub"/> + </xs:choice> + </xs:complexType> + </xs:element> + <xs:element name="MySub" abstract="true"/> + <xs:element name="MyBadSub" type="xs:string" block="substitution"/> + <xs:element name="innerValid" substitutionGroup="MySub" type="xs:string"/> + <xs:element name="innerInvalid" substitutionGroup="MyBadSub" type="xs:string"/> +</xs:schema> diff --git a/tests/auto/xmlpatternsvalidator/tst_xmlpatternsvalidator.cpp b/tests/auto/xmlpatternsvalidator/tst_xmlpatternsvalidator.cpp index fb94e09..19a20ba 100644 --- a/tests/auto/xmlpatternsvalidator/tst_xmlpatternsvalidator.cpp +++ b/tests/auto/xmlpatternsvalidator/tst_xmlpatternsvalidator.cpp @@ -228,6 +228,18 @@ void tst_XmlPatternsValidator::xsdSupport_data() const << (QStringList() << path + QLatin1String("dateTime-with-microseconds.xml") << path + QLatin1String("dateTime-with-microseconds.xsd")) << QString(); + + QTest::newRow("A document with a valid substitution group") + << 0 + << (QStringList() << path + QLatin1String("substitution-group-valid.xml") + << path + QLatin1String("substitution-group.xsd")) + << QString(); + + QTest::newRow("A document attempting to use a prohibited substitution") + << 1 + << (QStringList() << path + QLatin1String("substitution-group-invalid.xml") + << path + QLatin1String("substitution-group.xsd")) + << QString(); } QTEST_MAIN(tst_XmlPatternsValidator) |