[XERCESC-1350] DTDElementDecl lazy initialization not thread safe.

• Type: Bug
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Affects Version/s: 3.0.0, 3.0.1, 3.1.0, 3.1.1, 3.1.2, 3.2.0, 3.1.3, 3.1.4
• Component/s: Validating Parser (DTD)
• Environment: Solaris
• Assignee: Unassigned
• Reporter: Mark Weissman
• Created: 22/Feb/05 02:53
• Updated: 12/Jul/17 23:22

Description

DTDElementDecl.cpp && DTDElementDecl.hpp perform lazy evaluation on many
of the get() methods. This is used in cached grammars which may
be shared by multiple threads and accessed unsynchonized.

Lazy initialization is generally a bad pattern, but particularly in multithreaded
applications. It should be disabled for shared grammars.

I work around this by calling all the getters when I create my Grammar pool
but I think this should not be necessary.

It would be good if somebody could check all the lazy evaluations that might
affect shared grammars.