From: kfujino <kfujino@13f79535-47bb-0310-9956-ffa450edef68>
Date: Fri, 6 Aug 2010 05:56:21 +0000 (+0000)
Subject: Add Null check when CHANGE_SESSION_ID message received.
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=f1ceeaf0f048891d64bc7d4b38ace2ede8091114;p=tomcat7.0

Add Null check when CHANGE_SESSION_ID message received.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@982877 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/java/org/apache/catalina/ha/session/DeltaManager.java b/java/org/apache/catalina/ha/session/DeltaManager.java
index 90c28f1d6..59f0ecc34 100644
--- a/java/org/apache/catalina/ha/session/DeltaManager.java
+++ b/java/org/apache/catalina/ha/session/DeltaManager.java
@@ -1436,13 +1436,15 @@ public class DeltaManager extends ClusterManagerBase{
     protected void handleCHANGE_SESSION_ID(SessionMessage msg,Member sender) throws IOException {
         counterReceive_EVT_CHANGE_SESSION_ID++;
         DeltaSession session = (DeltaSession) findSession(msg.getSessionID());
-        String newSessionID = deserializeSessionId(msg.getSession());
-        session.setPrimarySession(false);
-        if(notifySessionListenersOnReplication) {
-            session.setId(newSessionID);
-        } else {
-            session.setIdInternal(newSessionID);
-            add(session);
+        if (session != null) {
+            String newSessionID = deserializeSessionId(msg.getSession());
+            session.setPrimarySession(false);
+            if (notifySessionListenersOnReplication) {
+                session.setId(newSessionID);
+            } else {
+                session.setIdInternal(newSessionID);
+                add(session);
+            }
         }
     }
 
diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml
index 166dd0626..8a02c4171 100644
--- a/webapps/docs/changelog.xml
+++ b/webapps/docs/changelog.xml
@@ -63,6 +63,9 @@
         If you send session to only same domain, use DomainFilterInterceptor. 
         (kfujino)
       </fix>
+      <fix>
+        Add Null check when CHANGE_SESSION_ID message received. (kfujino)
+      </fix>
     </changelog>
   </subsection>
   <subsection name="Web applications">