This is the mail archive of the cygwin-patches mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH 1/1] Cygwin: pty: Fix potential state mismatch regarding pseudo console.


On Sep  5 19:44, Takashi Yano wrote:
> - PTY with pseudo console support sitll has problem which potentially
>   cause state mismatch between state variable and real console state.
>   This patch fixes this issue.
> ---
>  winsup/cygwin/dtable.cc | 9 +++++++--
>  1 file changed, 7 insertions(+), 2 deletions(-)
> 
> diff --git a/winsup/cygwin/dtable.cc b/winsup/cygwin/dtable.cc
> index 4e9b6ed56..7b2e52005 100644
> --- a/winsup/cygwin/dtable.cc
> +++ b/winsup/cygwin/dtable.cc
> @@ -159,14 +159,19 @@ dtable::stdio_init ()
>  	    {
>  	      bool attached = !!fhandler_console::get_console_process_id
>  		(ptys->getHelperProcessId (), true);
> -	      if (!attached)
> +	      if (attached)
> +		break;
> +	      else
>  		{
>  		  /* Not attached to pseudo console in fork() or spawn()
>  		     by some reason. This happens if the executable is
>  		     a windows GUI binary, such as mintty. */
>  		  FreeConsole ();
>  		  if (AttachConsole (ptys->getHelperProcessId ()))
> -		    break;
> +		    {
> +		      ptys->fixup_after_attach (false);
> +		      break;
> +		    }
>  		}
>  	    }
>  	}
> -- 
> 2.21.0

Pushed.

FYI, Ken Brown is taking over from here as temporary co-maintainer for
Cygwin.  He'll build and upload Cygwin test releases during my absence.
Please don't hesitate to send more patches as required, Ken will review
and eventually push them.

Ken, if you like, please generate a new Cygwin test release now.


Thanks,
Corinna

-- 
Corinna Vinschen
Cygwin Maintainer

Attachment: signature.asc
Description: PGP signature


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]