donaldp 01/11/09 00:10:06
Modified: src/java/org/apache/avalon/excalibur/component
ThreadSafeComponentHandler.java
Log:
When a ThreadSafeComponentHandler has been created via the
ThreadSafeComponentHandler( final Component component ) constructor the
m_factory member is set to null. However, in setLogger(Logger log) this is not
checked for, and this results in a NullPointerException.
This patch fixes the problem:
PR: 4741
Submitted By: "Leo Sutic" <[EMAIL PROTECTED]>
Revision Changes Path
1.7 +5 -2
jakarta-avalon-excalibur/src/java/org/apache/avalon/excalibur/component/ThreadSafeComponentHandler.java
Index: ThreadSafeComponentHandler.java
===================================================================
RCS file:
/home/cvs/jakarta-avalon-excalibur/src/java/org/apache/avalon/excalibur/component/ThreadSafeComponentHandler.java,v
retrieving revision 1.6
retrieving revision 1.7
diff -u -r1.6 -r1.7
--- ThreadSafeComponentHandler.java 2001/11/01 14:25:57 1.6
+++ ThreadSafeComponentHandler.java 2001/11/09 08:10:06 1.7
@@ -21,7 +21,7 @@
* and destroyed correctly.
*
* @author <a href="mailto:[EMAIL PROTECTED]">Berin Loritsch</a>
- * @version CVS $Revision: 1.6 $ $Date: 2001/11/01 14:25:57 $
+ * @version CVS $Revision: 1.7 $ $Date: 2001/11/09 08:10:06 $
* @since 4.0
*/
public class ThreadSafeComponentHandler extends ComponentHandler {
@@ -60,7 +60,10 @@
public void setLogger(Logger log)
{
- m_factory.setLogger(log);
+ if ( this.m_factory != null )
+ {
+ m_factory.setLogger(log);
+ }
super.setLogger(log);
}
--
To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>