Author: markt
Date: Fri Mar  6 14:01:51 2009
New Revision: 750905

URL: http://svn.apache.org/viewvc?rev=750905&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/tc6.0.x/trunk/   (props changed)
    tomcat/tc6.0.x/trunk/STATUS.txt
    tomcat/tc6.0.x/trunk/java/org/apache/catalina/startup/HostConfig.java
    tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml

Propchange: tomcat/tc6.0.x/trunk/
------------------------------------------------------------------------------
--- svn:mergeinfo (original)
+++ svn:mergeinfo Fri Mar  6 14:01:51 2009
@@ -1 +1 @@
-/tomcat/trunk:601180,606992,612607,630314,640888,652744,653247,673796,673820,683982,684001,684081,684234,684269-684270,685177,687503,687645,689402,690781,691392,691805,692748,693378,694992,695053,695311,696780,696782,698012,698227,698236,698613,699427,699634,701355,709294,709811,709816,710063,710066,710125,710205,711126,711600,712461,712467,718360,719602,719626,719628,720046,720069,721040,723404,723738,726052,727303,728032,728768,728947,729057,729567,729569,729571,729809,729815,729934,730250,730590,731651,732859,732863,734734,740675,740684,742677,747834,748344
+/tomcat/trunk:601180,606992,612607,630314,640888,652744,653247,673796,673820,683982,684001,684081,684234,684269-684270,685177,687503,687645,689402,690781,691392,691805,692748,693378,694992,695053,695311,696780,696782,698012,698227,698236,698613,699427,699634,701355,709294,709811,709816,710063,710066,710125,710205,711126,711600,712461,712467,718360,719602,719626,719628,720046,720069,721040,723404,723738,726052,727303,728032,728768,728947,729057,729567,729569,729571,729809,729815,729934,730250,730590,731651,732859,732863,734734,740675,740684,742677,742697,747834,748344

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=750905&r1=750904&r2=750905&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Fri Mar  6 14:01:51 2009
@@ -95,13 +95,6 @@
       code a warning that it won't be there in the next version.
   -1: 
 
-* 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
-  http://svn.apache.org/viewvc?rev=742697&view=rev
-  +1: markt, remm, mturk, jim
-  -1: 
-
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=39396
   Don't include TRACEE in OPTIONS response unless we know it hasn't been
   disabled in the connector

Modified: tomcat/tc6.0.x/trunk/java/org/apache/catalina/startup/HostConfig.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/catalina/startup/HostConfig.java?rev=750905&r1=750904&r2=750905&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/java/org/apache/catalina/startup/HostConfig.java 
(original)
+++ tomcat/tc6.0.x/trunk/java/org/apache/catalina/startup/HostConfig.java Fri 
Mar  6 14:01:51 2009
@@ -1076,7 +1076,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);
@@ -1116,7 +1116,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);
@@ -1133,7 +1133,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/tc6.0.x/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml?rev=750905&r1=750904&r2=750905&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml Fri Mar  6 14:01:51 2009
@@ -54,6 +54,10 @@
         check for an exact match against the appBase. (markt)
       </fix>
       <fix>
+        <bug>39013</bug>: When testing for invalid docBase, test for an exact
+        match with the appBase dir. (markt)
+      </fix>
+      <fix>
         <bug>42747</bug>: Ensure context.xml takes effect on first deployment
         for WAR and DIR deployments. context.xml is now copied to
         CATALINA_BASE/&lt;engine name&gt;/&lt;host name&gt; for DIR as well as



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to