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]>

Reply via email to