Description Alexander Imfeld 2010-07-07 04:31:04 EDT

Build Identifier: 2.5.2

Reading report designs involves decoding base64 encoded values (in Base64PropertyState). Base64PropertyState has a static reference to apache commons Base64.

Base64 is no longer thread-safe in v1.4. As a consequence report design reading is not thread-safe when using commons-codec v1.4.

It would be nice to upgrade BIRT to allow for using commons-codec v1.4. I suggest to make the reference to Base64 in Base64PropertyState non-static.

Sample Stacktrace:
Caused by: java.lang.NullPointerException
	at org.apache.commons.codec.binary.Base64.decode(Base64.java:561)
	at org.apache.commons.codec.binary.Base64.decode(Base64.java:741)
	at org.eclipse.birt.report.model.parser.Base64PropertyState.getDecodedBase64Value(Base64PropertyState.java:114)
	at org.eclipse.birt.report.model.parser.Base64PropertyState.end(Base64PropertyState.java:87)
	at org.eclipse.birt.report.model.util.XMLParserHandler.endElement(XMLParserHandler.java:176)
	at org.apache.xerces.parsers.AbstractSAXParser.endElement(Unknown Source)
	at org.apache.xerces.impl.XMLDocumentFragmentScannerImpl.scanEndElement(Unknown Source)
	at org.apache.xerces.impl.XMLDocumentFragmentScannerImpl$FragmentContentDispatcher.dispatch(Unknown Source)
	at org.apache.xerces.impl.XMLDocumentFragmentScannerImpl.scanDocument(Unknown Source)
	at org.apache.xerces.parsers.XML11Configuration.parse(Unknown Source)
	at org.apache.xerces.parsers.XML11Configuration.parse(Unknown Source)
	at org.apache.xerces.parsers.XMLParser.parse(Unknown Source)
	at org.apache.xerces.parsers.AbstractSAXParser.parse(Unknown Source)
	at org.apache.xerces.jaxp.SAXParserImpl$JAXPSAXParser.parse(Unknown Source)
	at javax.xml.parsers.SAXParser.parse(SAXParser.java:395)
	at org.eclipse.birt.report.model.parser.ModuleReader.readModule(ModuleReader.java:94)
	at org.eclipse.birt.report.model.parser.DesignReader.read(DesignReader.java:88)
	at org.eclipse.birt.report.model.core.DesignSessionImpl.openDesign(DesignSessionImpl.java:314)
	at org.eclipse.birt.report.model.api.SessionHandleImpl.openDesign(SessionHandleImpl.java:294)
	at org.eclipse.birt.report.engine.parser.ReportParser.getDesignHandle(ReportParser.java:158)
	at org.eclipse.birt.report.engine.api.impl.ReportEngineHelper.getReportDesignHandle(ReportEngineHelper.java:254)
	at org.eclipse.birt.report.engine.api.impl.ReportEngineHelper.openReportDesign(ReportEngineHelper.java:273)
	at org.eclipse.birt.report.engine.api.impl.ReportEngineHelper.openReportDesign(ReportEngineHelper.java:183)
	at org.eclipse.birt.report.engine.api.impl.ReportEngineHelper.openReportDesign(ReportEngineHelper.java:160)
	at org.eclipse.birt.report.engine.api.impl.ReportEngine.openReportDesign(ReportEngine.java:333)


Reproducible: Always

Steps to Reproduce:
1. use apache commons-codec v1.4
2. have multiple threads simultaneously read report designs