Properly escape Windows command line arguments

Signed-off-by: James Couball <jcouball@yahoo.com>
ruby-git · Jan 3, 2022 · cca7109 · cca7109
1 parent 934d17d
commit cca7109
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/lib/git/lib.rb b/lib/git/lib.rb
@@ -1191,8 +1191,9 @@ def escape_for_sh(s)
     end
 
     def escape_for_windows(s)
-      # Windows does not need single quote escaping inside double quotes
-      %Q{"#{s}"}
+      # Escape existing double quotes in s and then wrap the result with double quotes
+      escaped_string = s.to_s.gsub('"','\\"')
+      %Q{"#{escaped_string}"}
     end
 
     def windows_platform?