[bugfix] Fix an issue with spaces in Windows file paths when locating conf.xml

Author: adamretter

exist-start/src/main/java/org/exist/start/Main.java

@@ -312,6 +312,7 @@ public void runEx(String[] args) throws StartException {
         Thread.currentThread().setContextClassLoader(cl);
 
         // Invoke main class using new classloader.
+        args = cleanupArgs(args);
         try {
             invokeMain(cl, _classname, args);
         } catch (final Exception e) {
@@ -320,6 +321,34 @@ public void runEx(String[] args) throws StartException {
         }
     }
 
+    public static String[] cleanupArgs(final String[] args) {
+        if (args == null) {
+            return null;
+        }
+        if (args.length == 0) {
+            return args;
+        }
+
+        final String[] nargs = new String[args.length];
+        for (int i = 0; i < args.length; i++) {
+            String arg = args[i];
+
+            if (arg.startsWith("\"") && arg.indexOf('\"', 1) > -1 && (arg.endsWith("conf.xml") || arg.endsWith("log4j2.xml"))) {
+                // String that starts with quote, has a quote somewhere in it, and ends with conf.xml or log4j2.xml - such invalid strings can be produced by startup.bat on Windows
+                arg = arg.replaceAll("\"", "");
+            }
+
+            if (arg.indexOf('\\') != -1 && arg.indexOf('\\') < arg.indexOf('/') && (arg.endsWith("conf.xml") || arg.endsWith("log4j2.xml"))) {
+                // String that contains both '\' path separators followed by `/` path separators, and ends with conf.xml or log4j2.xml - such invalid strings can be produced by startup.bat on Windows
+                arg = arg.replaceAll("/", "\\\\");
+            }
+
+            nargs[i] = arg;
+        }
+
+        return nargs;
+    }
+
     private Optional<String> getFromSysPropOrEnv(final String sysPropName, final String envVarName) {
         Optional<String> value = Optional.ofNullable(System.getProperty(sysPropName));
         if (!value.isPresent()) {