Author: markt
Date: Tue Mar 24 12:53:32 2009
New Revision: 757767

URL: http://svn.apache.org/viewvc?rev=757767&view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=39013
When testing for invalid docBase, test for an exact match with the appBase dir

Modified:
    
tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/startup/HostConfig.java
    tomcat/container/tc5.5.x/webapps/docs/changelog.xml

Modified: 
tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/startup/HostConfig.java
URL: 
http://svn.apache.org/viewvc/tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/startup/HostConfig.java?rev=757767&r1=757766&r2=757767&view=diff
==============================================================================
--- 
tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/startup/HostConfig.java
 (original)
+++ 
tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/startup/HostConfig.java
 Tue Mar 24 12:53:32 2009
@@ -1020,7 +1020,7 @@
                         try {
                             File current = new File(resources[j]);
                             current = current.getCanonicalFile();
-                            if 
((current.getAbsolutePath().startsWith(appBase().getAbsolutePath()))
+                            if 
((current.getAbsolutePath().startsWith(appBase().getAbsolutePath() + 
File.separator))
                                     || 
(current.getAbsolutePath().startsWith(configBase().getAbsolutePath()))) {
                                 if (log.isDebugEnabled())
                                     log.debug("Delete " + current);
@@ -1060,7 +1060,7 @@
                     try {
                         File current = new File(resources[j]);
                         current = current.getCanonicalFile();
-                        if 
((current.getAbsolutePath().startsWith(appBase().getAbsolutePath()))
+                        if 
((current.getAbsolutePath().startsWith(appBase().getAbsolutePath() + 
File.separator))
                             || 
(current.getAbsolutePath().startsWith(configBase().getAbsolutePath()))) {
                             if (log.isDebugEnabled())
                                 log.debug("Delete " + current);
@@ -1077,7 +1077,7 @@
                     try {
                         File current = new File(resources2[j]);
                         current = current.getCanonicalFile();
-                        if 
((current.getAbsolutePath().startsWith(appBase().getAbsolutePath()))
+                        if 
((current.getAbsolutePath().startsWith(appBase().getAbsolutePath() + 
File.separator))
                             || 
((current.getAbsolutePath().startsWith(configBase().getAbsolutePath())
                                  && 
(current.getAbsolutePath().endsWith(".xml"))))) {
                             if (log.isDebugEnabled())

Modified: tomcat/container/tc5.5.x/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/container/tc5.5.x/webapps/docs/changelog.xml?rev=757767&r1=757766&r2=757767&view=diff
==============================================================================
--- tomcat/container/tc5.5.x/webapps/docs/changelog.xml (original)
+++ tomcat/container/tc5.5.x/webapps/docs/changelog.xml Tue Mar 24 12:53:32 2009
@@ -35,6 +35,10 @@
   <subsection name="Catalina">
     <changelog>
       <fix>
+        <bug>39013</bug>: When testing for an invalid docBase, use an exact
+        match for the appBase. (markt)
+      </fix>
+      <fix>
         Remove wrong "No role found" realm debug log message,
         even if a role was found. (rjung)
       </fix>
@@ -42,6 +46,10 @@
         <bug>41407</bug>: JAAS Realm now works with CLIENT-CERT authentication.
         (markt)
       </fix>
+      <add>
+        <bug>42419</bug>: Add a system property that enables the name of the
+        session cookie and session path parameter to be configured. (markt)
+      </add>
       <fix>
         <bug>45576</bug>: JAAS Realm now works with DIGEST authentication.
         (markt)



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to