Francois Gouget : testbot/testagentd: Fix the console when redirecting stdin/out/err on Windows.

[Alexandre Julliard]

Module: tools
Branch: master
Commit: dbe7ab2a74b347e773a74d7076dc1dabc521f94f
URL:    https://source.winehq.org/git/tools.git/?a=commit;h=dbe7ab2a74b347e773a74d7076dc1dabc521f94f

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Thu Oct 31 17:03:23 2019 +0100

testbot/testagentd: Fix the console when redirecting stdin/out/err on Windows.

When When TestAgentd runs in --detach mode it does not have a console.
The child console processes still get a console but when using
STARTF_USESTDHANDLES the non-redirected stdin/out/err ifile handles
would not point to it. This appears to be because CreateProcess()
compares the STARTUPINFO handles with the parent's GetStdHandle() ones
and only reconnects them to the child console if they match.

So if the parent has no console the child inherits garbage values for
the GetStdHandle()s, that is GetFileType() returns FILE_TYPE_UNKNOWN for
those and their behavior is somewhat random. This would break some
kernel32:console STD_INPUT_HANDLE tests.

So when in --detach mode TestAgentd now creates a new console before
calling CreateProcess(). This resets stdin/out/err to point to the
console causing the child process to inherit proper handle values.
TestAgentd then detaches from the console which means it may just flash
on screen if starting a GUI application.

Increment the version so clients can check whether Run() redirections
are fully usable.

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 testbot/src/testagentd/platform_windows.c | 8 +++++++-
 testbot/src/testagentd/testagentd.c       | 1 +
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/testbot/src/testagentd/platform_windows.c b/testbot/src/testagentd/platform_windows.c
index ea6f03f..057e8fc 100644
--- a/testbot/src/testagentd/platform_windows.c
+++ b/testbot/src/testagentd/platform_windows.c
@@ -29,9 +29,10 @@ struct child_t
     DWORD pid;
     HANDLE handle;
 };
-
 static struct list children = LIST_INIT(children);
 
+static BOOLEAN detached = FALSE;
+
 
 uint64_t platform_run(char** argv, uint32_t flags, char** redirects)
 {
@@ -79,6 +80,8 @@ uint64_t platform_run(char** argv, uint32_t flags, char** redirects)
     *(d-1) = '\0';
 
     /* Prepare the redirections */
+    if (detached)
+        AllocConsole();
     has_redirects = 0;
     for (i = 0; i < 3; i++)
     {
@@ -149,6 +152,8 @@ uint64_t platform_run(char** argv, uint32_t flags, char** redirects)
     for (i = 0; i < 3; i++)
         if (redirects[i][0])
             CloseHandle(fhs[i]);
+    if (detached)
+        platform_detach_console();
 
     return pi.dwProcessId;
 }
@@ -692,6 +697,7 @@ void platform_detach_console(void)
     SetStdHandle(STD_INPUT_HANDLE, INVALID_HANDLE_VALUE);
     SetStdHandle(STD_OUTPUT_HANDLE, INVALID_HANDLE_VALUE);
     SetStdHandle(STD_ERROR_HANDLE, INVALID_HANDLE_VALUE);
+    detached = TRUE;
 }
 
 int platform_init(void)
diff --git a/testbot/src/testagentd/testagentd.c b/testbot/src/testagentd/testagentd.c
index eb93aa2..994f7c2 100644
--- a/testbot/src/testagentd/testagentd.c
+++ b/testbot/src/testagentd/testagentd.c
@@ -41,6 +41,7 @@
  * 1.7:  Add --show-restarts and the setproperty RPC.
  * 1.8:  Add the restart RPC, server.arg* properties, fix upgrades if >1
  *       server.
+ * 1.9:  Fix the console handling when redirecting stdin/out/err on Windows.
  */
 #define PROTOCOL_VERSION "testagentd 1.8"