Author: remy.maucherat(a)jboss.com Date: 2014-11-18 12:31:39 -0500 (Tue, 18 Nov 2014) New Revision: 2545 Modified: branches/7.5.x/src/main/java/org/apache/catalina/valves/CrawlerSessionManagerValve.java Log: BZ1163126 refix: The inner class used was not actually serializable (additional stuff). Modified: branches/7.5.x/src/main/java/org/apache/catalina/valves/CrawlerSessionManagerValve.java =================================================================== --- branches/7.5.x/src/main/java/org/apache/catalina/valves/CrawlerSessionManagerValve.java 2014-11-18 15:58:06 UTC (rev 2544) +++ branches/7.5.x/src/main/java/org/apache/catalina/valves/CrawlerSessionManagerValve.java 2014-11-18 17:31:39 UTC (rev 2545) @@ -241,9 +241,9 @@ } class CrawlerBindingListener implements HttpSessionBindingListener, Serializable { - private static final long serialVersionUID = -3775762684177732270L; - private final transient Map<String,String> clientIpSessionId; - private final transient Map<String,String> sessionIdClientIp; + private static final long serialVersionUID = -8841692120840734349L; + private transient Map<String,String> clientIpSessionId; + private transient Map<String,String> sessionIdClientIp; CrawlerBindingListener(Map<String,String> clientIpSessionId, Map<String,String> sessionIdClientIp) { this.clientIpSessionId = clientIpSessionId; @@ -257,9 +257,11 @@ @Override public void valueUnbound(HttpSessionBindingEvent event) { - String clientIp = sessionIdClientIp.remove(event.getSession().getId()); - if (clientIp != null) { - clientIpSessionId.remove(clientIp); + if (sessionIdClientIp != null) { + String clientIp = sessionIdClientIp.remove(event.getSession().getId()); + if (clientIp != null) { + clientIpSessionId.remove(clientIp); + } } } }