From 34cc372abe5dfd1d868b1e4428c8b5f49cbda5ce Mon Sep 17 00:00:00 2001 From: Christopher Faylor Date: Sat, 26 Jul 2008 19:35:21 +0000 Subject: [PATCH] * fhandler_tty.cc (fhandler_pty_master::setup): Reorganize so that all operations for each end of the pipe are grouped together. --- winsup/cygwin/ChangeLog | 5 +++++ winsup/cygwin/fhandler_tty.cc | 38 ++++++++++++++++------------------- 2 files changed, 22 insertions(+), 21 deletions(-) diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog index 244da549e..425de1085 100644 --- a/winsup/cygwin/ChangeLog +++ b/winsup/cygwin/ChangeLog @@ -1,3 +1,8 @@ +2008-07-26 Christopher Faylor + + * fhandler_tty.cc (fhandler_pty_master::setup): Reorganize so that all + operations for each end of the pipe are grouped together. + 2008-07-25 Corinna Vinschen * dcrt0.cc (dll_crt0_1): Call malloc_init and user_shared_initialize_1 diff --git a/winsup/cygwin/fhandler_tty.cc b/winsup/cygwin/fhandler_tty.cc index ab52d6dfe..d4b710d8f 100644 --- a/winsup/cygwin/fhandler_tty.cc +++ b/winsup/cygwin/fhandler_tty.cc @@ -1362,7 +1362,7 @@ fhandler_pty_master::setup (bool ispty) char pipename[sizeof("ttyNNNN-from-master")]; __small_sprintf (pipename, "tty%d-from-master", get_unit ()); - res = fhandler_pipe::create_selectable (&sec_all, from_master, + res = fhandler_pipe::create_selectable (&sec_none_nih, from_master, get_output_handle (), 128 * 1024, pipename); if (res) @@ -1370,13 +1370,9 @@ fhandler_pty_master::setup (bool ispty) errstr = "input pipe"; goto err; } - - __small_sprintf (pipename, "tty%d-to-master", get_unit ()); - res = fhandler_pipe::create_selectable (&sec_all, get_io_handle (), - to_master, 128 * 1024, pipename); - if (res) + if (!SetHandleInformation (get_output_handle (), HANDLE_FLAG_INHERIT, HANDLE_FLAG_INHERIT)) { - errstr = "output pipe"; + errstr = "inheritable get_output_handle ()"; goto err; } @@ -1384,6 +1380,20 @@ fhandler_pty_master::setup (bool ispty) termios_printf ("can't set output_handle(%p) to non-blocking mode", get_output_handle ()); + __small_sprintf (pipename, "tty%d-to-master", get_unit ()); + res = fhandler_pipe::create_selectable (&sec_none_nih, get_io_handle (), + to_master, 128 * 1024, pipename); + if (res) + { + errstr = "output pipe"; + goto err; + } + if (!SetHandleInformation (get_io_handle (), HANDLE_FLAG_INHERIT, HANDLE_FLAG_INHERIT)) + { + errstr = "inheritable get_io_handle ()"; + goto err; + } + need_nl = 0; /* We do not allow others to open us (for handle duplication) @@ -1419,20 +1429,6 @@ fhandler_pty_master::setup (bool ispty) if (!(input_mutex = CreateMutex (&sec_all, FALSE, buf))) goto err; - if (!DuplicateHandle (hMainProc, from_master, hMainProc, &from_master, 0, false, - DUPLICATE_SAME_ACCESS | DUPLICATE_CLOSE_SOURCE)) - { - errstr = "non-inheritable from_master"; - goto err; - } - - if (!DuplicateHandle (hMainProc, to_master, hMainProc, &to_master, 0, false, - DUPLICATE_SAME_ACCESS | DUPLICATE_CLOSE_SOURCE)) - { - errstr = "non-inheritable to_master"; - goto err; - } - t.from_master = from_master; t.to_master = to_master; // /* screws up tty master */ ProtectHandle1INH (output_mutex, output_mutex);