remm        01/07/10 15:08:30

  Modified:    catalina/src/share/org/apache/catalina/session
                        LocalStrings.properties StandardSession.java
  Log:
  - Specifying a null name will result in a illegal arg exception being thrown.
    Bug reported by Kumar Mettu <mettu.kumar at redspark.com>
  
  Revision  Changes    Path
  1.8       +1 -0      
jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/session/LocalStrings.properties
  
  Index: LocalStrings.properties
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/session/LocalStrings.properties,v
  retrieving revision 1.7
  retrieving revision 1.8
  diff -u -r1.7 -r1.8
  --- LocalStrings.properties   2001/04/26 01:40:55     1.7
  +++ LocalStrings.properties   2001/07/10 22:08:25     1.8
  @@ -42,6 +42,7 @@
   standardSession.sessionEvent=Session event listener threw exception
   standardSession.setAttribute.ise=setAttribute: Non-serializable attribute
   standardSession.setAttribute.ise=setAttribute: Session already invalidated
  +standardSession.setAttribute.namenull=setAttribute: name parameter cannot be null
   standardSession.sessionCreated=Created Session id = {0}
   persistentManager.loading=Loading {0} persisted sessions
   persistentManager.unloading=Saving {0} persisted sessions
  
  
  
  1.22      +9 -4      
jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/session/StandardSession.java
  
  Index: StandardSession.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/session/StandardSession.java,v
  retrieving revision 1.21
  retrieving revision 1.22
  diff -u -r1.21 -r1.22
  --- StandardSession.java      2001/07/10 03:18:57     1.21
  +++ StandardSession.java      2001/07/10 22:08:27     1.22
  @@ -1,7 +1,7 @@
   /*
  - * $Header: 
/home/cvs/jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/session/StandardSession.java,v
 1.21 2001/07/10 03:18:57 jon Exp $
  - * $Revision: 1.21 $
  - * $Date: 2001/07/10 03:18:57 $
  + * $Header: 
/home/cvs/jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/session/StandardSession.java,v
 1.22 2001/07/10 22:08:27 remm Exp $
  + * $Revision: 1.22 $
  + * $Date: 2001/07/10 22:08:27 $
    *
    * ====================================================================
    *
  @@ -111,7 +111,7 @@
    * @author Craig R. McClanahan
    * @author Sean Legassick
    * @author <a href="mailto:[EMAIL PROTECTED]";>Jon S. Stevens</a>
  - * @version $Revision: 1.21 $ $Date: 2001/07/10 03:18:57 $
  + * @version $Revision: 1.22 $ $Date: 2001/07/10 22:08:27 $
    */
   
   class StandardSession
  @@ -1021,6 +1021,11 @@
        *  invalidated session
        */
       public void setAttribute(String name, Object value) {
  +
  +        // Name cannot be null
  +        if (name == null)
  +            throw new IllegalArgumentException
  +                (sm.getString("standardSession.setAttribute.namenull"));
   
           // Null value is the same as removeAttribute()
           if (value == null) {
  
  
  

Reply via email to