[XERCESC-1626] loadGrammar is releasing a DOMBuilder previously parsed document when it shouldn't

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.7.0
• Fix Version/s: 2.8.0
• Component/s: DOM, Validating Parser (XML Schema)
• Environment: Windows XP Pro 32bit, ; Xerces-2.7.0,; Visual C++ 6.0 sp5
• Assignee: Unassigned
• Reporter: Ori D.
• Created: 22/Aug/06 20:34
• Updated: 12/Mar/18 04:05
• Resolved: 23/Aug/06 09:37

Description

I'm parsing xml file using Xerces-c 2.7 DOM.
After loading a schema grammar to the parser, my previously parsed xml document is released (and therefore further access to the document in the code causes segmentation fault)

I do the following things (more or less):

1. create a XMLGrammarPoolImpl object and then create a new
DOMBuilder using it as an input.

m_grammarPool = new XMLGrammarPoolImpl(XMLPlatformUtils::fgMemoryManager);
const XMLCh gLS[] =

{ chLatin_L, chLatin_S, chNull }

;
m_DomImpl = DOMImplementationRegistry::getDOMImplementation(gLS);
m_DomParser = ((DOMImplementationLS*)m_DomImpl)->createDOMBuilder(DOMImplementationLS::MODE_SYNCHRONOUS, 0, XMLPlatformUtils::fgMemoryManager, m_grammarPool);
m_DomParser->setFeature(XMLUni::fgDOMNamespaces, true);
m_DomParser->setFeature(XMLUni::fgXercesSchema, true);
m_DomParser->setFeature(XMLUni::fgXercesSchemaFullChecking, true);
m_DomParser->setFeature(XMLUni::fgDOMValidateIfSchema, true);
m_DomParser->setFeature(XMLUni::fgDOMDatatypeNormalization, true);

2. Parse an xml file using the DOMBuilder and keep a reference to
the DOMDocument, 'doc'

doc = m_DomParser->parse(inputSource);

3. Create a new DOMInputSource and call DOMBuilder
->loadGrammar(inputSrc, SchemaGrammarType, true)

SchemaGrammar pGrammar = (SchemaGrammar) m_DomParser->loadGrammar(schemaInputSource, Grammar::SchemaGrammarType, true);

The result is that my previous parsed document, 'doc' is being released by the following code in parsers\DOMBuilderImpl.cpp on DOMBuilderImpl::loadGrammar (line 703):

// Release DOM tree - DTD

DOMDocument* doc = adoptDocument();

if (doc)

doc->release();