Merge lp:~zorba-coders/zorba/bug867156-importsegfaults into lp:zorba

[Cezar Andrei]

According to Xerces-C++ team( http://issues.apache.org/jira/browse/XERCESC-1980 ) XMLSchema.xsd i.e. schema4schema should be avoided to be imported. This fix avoids importing this schema file.

[Cezar Andrei (cezar-andrei)]

review: Approve

[Chris Hillery (ceejatec)]

Could you add a URL in the code comment pointing to a Xerces mailing list
discussion or other page where it is mentioned that you shouldn't import
schema for schema? I suspect that is something that will surprise anyone
who looks at the Zorba source code later, so it would be good to have
third-party confirmation.

Ceej
aka Chris Hillery
On Apr 23, 2012 1:48 PM, "Cezar Andrei" <email address hidden> wrote:

> The proposal to merge lp:~zorba-coders/zorba/bug867156-importsegfaults
> into lp:zorba has been updated.
>
> Status: Needs review => Approved
>
> For more details, see:
>
> https://code.launchpad.net/~zorba-coders/zorba/bug867156-importsegfaults/+merge/103169
> --
>
> https://code.launchpad.net/~zorba-coders/zorba/bug867156-importsegfaults/+merge/103169
> You are requested to review the proposed merge of
> lp:~zorba-coders/zorba/bug867156-importsegfaults into lp:zorba.
>

[lp:~zorba-coders/zorba/bug867156-importsegfaults]

10787. By Cezar Andrei <email address hidden> on 2012-04-23

Added a comment with link to Xecres-C++ bug.

[Matthias Brantner (matthias-brantner)]

There are already plenty of definitions for the XMLSchema URI. I think it would make sense to reuse the one from src/zorbamisc/ns_consts.h.

review: Needs Fixing

Unmerged revisions

=== modified file 'src/compiler/translator/translator.cpp'
--- src/compiler/translator/translator.cpp	2012-04-23 10:46:38 +0000
+++ src/compiler/translator/translator.cpp	2012-04-23 20:45:24 +0000
@@ -105,7 +105,7 @@
 
 
 #define NODE_SORT_OPT
-
+#define XS_URI "http://www.w3.org/2001/XMLSchema"
 
 namespace zorba
 {
@@ -2011,6 +2011,13 @@
       theSctx->bind_ns(pfx, targetNS, loc, err::XQST0033);
   }
 
+  zstring xsdTNS = zstring(XS_URI);
+  if ( xsdTNS.compare(targetNS)==0 )
+  {
+    // Xerces doesn't like importing XMLSchema.xsd schema4schema, so we skip it
+    return no_state;
+  }
+
   store::Item_t targetNSItem = NULL;
   zstring tmp = targetNS;
   ITEM_FACTORY->createAnyURI(targetNSItem, tmp);