You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by bi...@apache.org on 2002/08/06 07:22:16 UTC

cvs commit: jakarta-tomcat/src/share/org/apache/jasper/runtime PageContextImpl.java

billbarker    2002/08/05 22:22:16

  Modified:    src/share/org/apache/jasper/runtime PageContextImpl.java
  Log:
  Make certain that we have a session before trying to remove attributes from it.
  
  Fix for bug #11463
  Reported By: Kubo Hiroshi hiroshi@netird.ad.jp
  
  Revision  Changes    Path
  1.23      +4 -2      jakarta-tomcat/src/share/org/apache/jasper/runtime/PageContextImpl.java
  
  Index: PageContextImpl.java
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat/src/share/org/apache/jasper/runtime/PageContextImpl.java,v
  retrieving revision 1.22
  retrieving revision 1.23
  diff -u -r1.22 -r1.23
  --- PageContextImpl.java	7 Sep 2001 04:39:31 -0000	1.22
  +++ PageContextImpl.java	6 Aug 2002 05:22:16 -0000	1.23
  @@ -118,7 +118,7 @@
   	    request=req;
   	    response=res;
   	    errorPageURL=err;
  -	    needsSession=n;;
  +	    needsSession=n;
   	    bufferSize=b;
   	    autoFlush=a;
   	}
  @@ -367,7 +367,9 @@
   	try {
   	    removeAttribute(name, PAGE_SCOPE);
   	    removeAttribute(name, REQUEST_SCOPE);
  -	    removeAttribute(name, SESSION_SCOPE);
  +	    if( session != null ) {
  +		removeAttribute(name, SESSION_SCOPE);
  +	    }
   	    removeAttribute(name, APPLICATION_SCOPE);
   	} catch (Exception ex) {
   	    // we remove as much as we can, and
  
  
  

--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>