From d7550d8322ab787f65ea74f026b2be3bf1c32adb Mon Sep 17 00:00:00 2001
From: uta <unknown>
Date: Fri, 8 Mar 2013 13:35:15 +0400
Subject: [PATCH] 8009463: Regression test
 test\java\lang\Runtime\exec\ArgWithSpaceAndFinalBackslash.java failing.
 Reviewed-by: alanb, ahgross

---
 src/windows/classes/java/lang/ProcessImpl.java | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/src/windows/classes/java/lang/ProcessImpl.java b/src/windows/classes/java/lang/ProcessImpl.java
index 87c89bb11..5f772aee8 100644
--- a/src/windows/classes/java/lang/ProcessImpl.java
+++ b/src/windows/classes/java/lang/ProcessImpl.java
@@ -263,6 +263,22 @@ final class ProcessImpl extends Process {
             if (needsEscaping(isCmdFile, s)) {
                 cmdbuf.append('"');
                 cmdbuf.append(s);
+
+                // The code protects the [java.exe] and console command line
+                // parser, that interprets the [\"] combination as an escape
+                // sequence for the ["] char.
+                //     http://msdn.microsoft.com/en-us/library/17w5ykft.aspx
+                //
+                // If the argument is an FS path, doubling of the tail [\]
+                // char is not a problem for non-console applications.
+                //
+                // The [\"] sequence is not an escape sequence for the [cmd.exe]
+                // command line parser. The case of the [""] tail escape
+                // sequence could not be realized due to the argument validation
+                // procedure.
+                if (!isCmdFile && s.endsWith("\\")) {
+                    cmdbuf.append('\\');
+                }
                 cmdbuf.append('"');
             } else {
                 cmdbuf.append(s);
-- 
GitLab