Attachment 32374 Details for Bug 57455 – Fix with tests

Fix with tests

bootstrap.diff (text/plain), 1.17 KB, created by Tobias Stoeckmann on 2015-01-17 20:28:59 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Tobias Stoeckmann

Created: 2015-01-17 20:28:59 UTC

Size: 1.17 KB

patch

obsolete

>Index: test/org/apache/catalina/startup/TestBootstrap.java
>===================================================================
>--- test/org/apache/catalina/startup/TestBootstrap.java	(revision 1652661)
>+++ test/org/apache/catalina/startup/TestBootstrap.java	(working copy)
>@@ -29,11 +29,16 @@
>     }
> 
>     @Test
>-    public void testOneNonQuoted() {
>+    public void testOneNonQuoted01() {
>         doTest("a", "a");
>     }
> 
>     @Test
>+    public void testOneNonQuoted02() {
>+        doTest("\"a", "\"a");
>+    }
>+
>+    @Test
>     public void testTwoNonQuoted01() {
>         doTest("a,b", "a", "b");
>     }
>Index: java/org/apache/catalina/startup/Bootstrap.java
>===================================================================
>--- java/org/apache/catalina/startup/Bootstrap.java	(revision 1652661)
>+++ java/org/apache/catalina/startup/Bootstrap.java	(working copy)
>@@ -569,7 +569,7 @@
> 
>             path = path.trim();
> 
>-            if (path.startsWith("\"") && path.length() > 1) {
>+            if (path.startsWith("\"") && path.endsWith("\"") && path.length() > 1) {
>                 path = path.substring(1, path.length() - 1);
>                 path = path.trim();
>             }

Actions: View

Attachments on bug 57455: 32374