Force full path exename when calling CreateProcess from cmd. Fix for #6.

gerardog · Jan 13, 2020 · 0a565cf · 0a565cf
1 parent 967751e
commit 0a565cf
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/src/gsudo.Tests/CmdTests.cs b/src/gsudo.Tests/CmdTests.cs
@@ -159,5 +159,17 @@ public void Cmd_UnexistentAppTest()
             p.WaitForExit();
             Assert.AreNotEqual(0, p.ExitCode);
         }
+
+        [TestMethod]
+        public void Cmd_BatchFileWithoutExtensionTest()
+        {
+            File.WriteAllText("HelloWorld.bat", "@echo Hello");
+
+            var p = new TestProcess("gsudo.exe", "HelloWorld");
+            p.WaitForExit();
+            Assert.AreEqual(string.Empty, p.GetStdErr());
+            Assert.AreEqual("Hello\r\n", p.GetStdOut());
+            Assert.AreEqual(0, p.ExitCode);
+        }
     }
 }
diff --git a/src/gsudo/Helpers/ArgumentsHelper.cs b/src/gsudo/Helpers/ArgumentsHelper.cs
@@ -65,6 +65,7 @@ internal static string[] AugmentCommand(string[] args)
                 }
                 else
                 {
+                    args[0] = exename; // Batch files not started by create process if no extension is specified.
                     return args;
                 }
             }