[PATCH] Cygwin: pty: Fix a bug in free_attached_console().
Corinna Vinschen
corinna-cygwin@cygwin.com
Thu May 28 07:27:13 GMT 2020
On May 28 12:43, Takashi Yano via Cygwin-patches wrote:
> - After commit 7659ff0f5afd751f42485f2684c799c5f37b0fb9, nohup does
> not work as expected. This patch fixes the issue.
>
> Addresses:
> https://cygwin.com/pipermail/cygwin-developers/2020-May/011885.html
> ---
> winsup/cygwin/fhandler_tty.cc | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/winsup/cygwin/fhandler_tty.cc b/winsup/cygwin/fhandler_tty.cc
> index df08dd20a..f29a2c214 100644
> --- a/winsup/cygwin/fhandler_tty.cc
> +++ b/winsup/cygwin/fhandler_tty.cc
> @@ -188,7 +188,10 @@ set_ishybrid_and_switch_to_pcon (HANDLE h)
> inline void
> fhandler_pty_slave::free_attached_console ()
> {
> - if (freeconsole_on_close && get_minor () == pcon_attached_to)
> + bool attached = get_ttyp () ?
> + fhandler_console::get_console_process_id (get_helper_process_id (), true)
> + : (get_minor () == pcon_attached_to);
> + if (freeconsole_on_close && attached)
> {
> FreeConsole ();
> pcon_attached_to = -1;
> --
> 2.26.2
Pushed.
Thanks,
Corinna
--
Corinna Vinschen
Cygwin Maintainer
More information about the Cygwin-patches
mailing list