You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by kk...@apache.org on 2010/10/29 01:58:00 UTC
svn commit: r1028554 - in /tomcat/tc6.0.x/trunk: STATUS.txt
java/org/apache/catalina/valves/PersistentValve.java
webapps/docs/changelog.xml
Author: kkolinko
Date: Thu Oct 28 23:58:00 2010
New Revision: 1028554
URL: http://svn.apache.org/viewvc?rev=1028554&view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=50131
Avoid possible NPE in debug output in PersistentValve.
Based on the patch provided by sebb
Modified:
tomcat/tc6.0.x/trunk/STATUS.txt
tomcat/tc6.0.x/trunk/java/org/apache/catalina/valves/PersistentValve.java
tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml
Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=1028554&r1=1028553&r2=1028554&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Thu Oct 28 23:58:00 2010
@@ -258,13 +258,6 @@ PATCHES PROPOSED TO BACKPORT:
this makes sense.
-1:
-* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=50131
- Avoid possible NPE in debug output in PersistentValve.
- Patch by sebb
- https://issues.apache.org/bugzilla/attachment.cgi?id=26194
- +1: kkolinko, kfujino, markt
- -1:
-
* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=50140
Don't ignore user specified installation directory on 64-bit platforms
Allow 32-bit and 64-bit JDKs to be selected on 64-bit platforms
Modified: tomcat/tc6.0.x/trunk/java/org/apache/catalina/valves/PersistentValve.java
URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/catalina/valves/PersistentValve.java?rev=1028554&r1=1028553&r2=1028554&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/java/org/apache/catalina/valves/PersistentValve.java (original)
+++ tomcat/tc6.0.x/trunk/java/org/apache/catalina/valves/PersistentValve.java Thu Oct 28 23:58:00 2010
@@ -178,9 +178,11 @@ public class PersistentValve
} else {
if (container.getLogger().isDebugEnabled())
container.getLogger().debug("newsessionId store: " + store + " session: " +
- session + " valid: " + session.isValid() +
- " Staled: " +
- isSessionStale(session, System.currentTimeMillis()));
+ session +
+ (session == null ? "" :
+ " valid: " + session.isValid() +
+ " stale: " +
+ isSessionStale(session, System.currentTimeMillis())));
}
} else {
Modified: tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml
URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml?rev=1028554&r1=1028553&r2=1028554&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml Thu Oct 28 23:58:00 2010
@@ -138,6 +138,10 @@
when running with a security manager. (kkolinko)
</fix>
<fix>
+ <bug>50131</bug>: Avoid possible NPE in debug output in PersistentValve.
+ Patch provided by sebb. (kkolinko)
+ </fix>
+ <fix>
<bug>50138</bug>: Fix threading issues in
<code>org.apache.catalina.security.SecurityUtil</code>. (markt)
</fix>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org