server: Use negative values for console pseudo handles.

Signed-off-by: Eric Pouech <eric.pouech@gmail.com> Signed-off-by: Jacek Caban <jacek@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

server: Use negative values for console pseudo handles.
95202ea4 · Eric Pouech · Alexandre Julliard · b4d91adb · 95202ea4 · 95202ea4
Commit 95202ea4 authored Apr 11, 2022 by Eric Pouech Committed by Alexandre Julliard Apr 12, 2022
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 5 deletions

condrv.h include/wine/condrv.h +3 -3

process.c server/process.c +2 -2

No files found.
--- a/include/wine/condrv.h
+++ b/include/wine/condrv.h
@@ -188,8 +188,8 @@ struct condrv_ctrl_event
 };

 /* Wine specific values for console inheritance (params->ConsoleHandle) */
-#define CONSOLE_HANDLE_ALLOC            UlongToHandle(1)
-#define CONSOLE_HANDLE_ALLOC_NO_WINDOW  UlongToHandle(2)
-#define CONSOLE_HANDLE_SHELL            UlongToHandle(3)
+#define CONSOLE_HANDLE_ALLOC            LongToHandle(-1)
+#define CONSOLE_HANDLE_ALLOC_NO_WINDOW  LongToHandle(-2)
+#define CONSOLE_HANDLE_SHELL            LongToHandle(-3)

 #endif /* _INC_CONDRV */
--- a/server/process.c
+++ b/server/process.c
@@ -1333,8 +1333,8 @@ DECL_HANDLER(new_process)
    /* connect to the window station */
    connect_process_winstation( process, parent_thread, parent );

-    /* set the process console */
-    if (info->data->console > 3)
+    /* inherit the process console, but keep pseudo handles (< 0), and 0 (= not attached to a console) as is */
+    if ((int)info->data->console > 0)
        info->data->console = duplicate_handle( parent, info->data->console, process,
                                                0, 0, DUPLICATE_SAME_ACCESS );