From: markt <markt@13f79535-47bb-0310-9956-ffa450edef68>
Date: Sun, 3 Jul 2011 13:15:58 +0000 (+0000)
Subject: Fix possible NPE
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=e7a22712f06cba1ff6cbb7640d391eba044bbb26;p=tomcat7.0

Fix possible NPE

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

diff --git a/java/org/apache/catalina/valves/CrawlerSessionManagerValve.java b/java/org/apache/catalina/valves/CrawlerSessionManagerValve.java
index 965a2bb9b..bf4268cbc 100644
--- a/java/org/apache/catalina/valves/CrawlerSessionManagerValve.java
+++ b/java/org/apache/catalina/valves/CrawlerSessionManagerValve.java
@@ -27,6 +27,7 @@ import javax.servlet.http.HttpSession;
 import javax.servlet.http.HttpSessionBindingEvent;
 import javax.servlet.http.HttpSessionBindingListener;
 
+import org.apache.catalina.LifecycleException;
 import org.apache.catalina.connector.Request;
 import org.apache.catalina.connector.Response;
 import org.apache.juli.logging.Log;
@@ -114,6 +115,14 @@ public class CrawlerSessionManagerValve extends ValveBase
 
 
     @Override
+    protected void initInternal() throws LifecycleException {
+        super.initInternal();
+        
+        uaPattern = Pattern.compile(crawlerUserAgents);
+    }
+
+
+    @Override
     public void invoke(Request request, Response response) throws IOException,
             ServletException {
 
diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml
index 0798f2862..2e8efcb05 100644
--- a/webapps/docs/changelog.xml
+++ b/webapps/docs/changelog.xml
@@ -54,6 +54,15 @@
   issues to not "pop up" wrt. others).
 -->
 <section name="Tomcat 7.0.18 (markt)">
+  <subsection name="Catalina">
+    <changelog>
+      <fix>
+        Correct regression introduced in 7.0.17 that triggered an NPE if a
+        CrawlerSessionManagerValve was used without setting crawlerUserAgents.
+        (markt)
+      </fix>
+    </changelog>
+  </subsection>
   <subsection name="Other">
     <changelog>
       <fix>