After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 455953 - XML Schema : internal validation error if there are IDCs in imported schemas
XML Schema : internal validation error if there are IDCs in imported schemas
Status: RESOLVED FIXED
Product: libxml2
Classification: Platform
Component: general
2.6.29
Other All
: Normal normal
: ---
Assigned To: Daniel Veillard
libxml QA maintainers
Depends on:
Blocks:
 
 
Reported: 2007-07-11 16:07 UTC by Frank Gross
Modified: 2007-08-23 09:31 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Test case (1.34 KB, application/zip)
2007-07-11 16:08 UTC, Frank Gross
  Details
Proposed patch (1.32 KB, patch)
2007-07-11 16:09 UTC, Frank Gross
none Details | Review

Description Frank Gross 2007-07-11 16:07:36 UTC
Validation of XML documents with a XML Schema containing IDCs and importing other XML schemas with IDC inside, generates an internal error.

See attached test case that produces following output:

$  xmllint --schema MainSchema.xsd Main.xml

Main.xml:4: element AnotherElement: Schemas validity error : Internal error: xmlSchemaIDCRegisterMatchers, Could not find an augmented IDC item for an IDC definition.
Main.xml:4: element AnotherElement: Schemas validity error : Internal error: xmlSchemaValidateElem, calling xmlSchemaValidateElemDecl().
Main.xml:4: element AnotherElement: Schemas validity error : Internal error: xmlSchemaDocWalk, calling xmlSchemaValidateElem().
Main.xml validation generated an internal error

And take a look at proposed patch.
Comment 1 Frank Gross 2007-07-11 16:08:48 UTC
Created attachment 91631 [details]
Test case
Comment 2 Frank Gross 2007-07-11 16:09:55 UTC
Created attachment 91633 [details] [review]
Proposed patch
Comment 3 Daniel Veillard 2007-08-23 09:31:18 UTC
Okay, the patch looks sane, and seems to fix the problem, and doesn't
introduce any detectable regressions. So I applied it added the test to
the regression suite and commited it to SVN, 

   thanks a lot !

Daniel