larryi      01/09/17 08:09:33

  Modified:    src/facade22/org/apache/tomcat/facade HttpSessionFacade.java
  Log:
  Add a check for null in case two requests try to remove an attribute
  simultaneously.
  
  Revision  Changes    Path
  1.14      +5 -3      
jakarta-tomcat/src/facade22/org/apache/tomcat/facade/HttpSessionFacade.java
  
  Index: HttpSessionFacade.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat/src/facade22/org/apache/tomcat/facade/HttpSessionFacade.java,v
  retrieving revision 1.13
  retrieving revision 1.14
  diff -u -r1.13 -r1.14
  --- HttpSessionFacade.java    2001/09/17 04:56:02     1.13
  +++ HttpSessionFacade.java    2001/09/17 15:09:33     1.14
  @@ -271,9 +271,11 @@
                // double check ( probably not needed since setAttribute calls
                // remove if it detects a value
                object=realSession.getAttribute( name );
  -             realSession.removeAttribute(name);
  -             ((HttpSessionBindingListener) object).valueUnbound
  -                 (new HttpSessionBindingEvent( this, name));
  +             if( object != null ) {
  +                 realSession.removeAttribute(name);
  +                 ((HttpSessionBindingListener) object).valueUnbound
  +                     (new HttpSessionBindingEvent( this, name));
  +             }
            }
        } else {
            // Regular object, no sync overhead
  
  
  

Reply via email to