Re: [PATCH v5 0/1] Fix PTY state management in pseudo console support.

[Corinna Vinschen <corinna-cygwin at cygwin dot com>]

On Sep  4 10:45, Takashi Yano wrote:
> Pseudo console support in test release TEST: Cygwin 3.1.0-0.3,
> introduced by commit 169d65a5774acc76ce3f3feeedcbae7405aa9b57,
> has some bugs which cause mismatch between state variables and
> real pseudo console state regarding console attaching and r/w
> pipe switching. This patch fixes this issue by redesigning the
> state management.
> 
> v5:
> Revise based on
> https://cygwin.com/ml/cygwin-patches/2019-q3/msg00111.html
> 
> v4:
> Small bug fix again.
> 
> v3:
> Fix the first issue (Bad file descriptor) reported in
> https://cygwin.com/ml/cygwin-patches/2019-q3/msg00104.html
> 
> v2:
> Small bug fixed from v1.
> 
> Takashi Yano (1):
>   Cygwin: pty: Fix state management for pseudo console support.
> 
>  winsup/cygwin/dtable.cc           |  38 +--
>  winsup/cygwin/fhandler.h          |   6 +-
>  winsup/cygwin/fhandler_console.cc |  25 +-
>  winsup/cygwin/fhandler_tty.cc     | 385 ++++++++++++++++--------------
>  winsup/cygwin/fork.cc             |  24 +-
>  winsup/cygwin/spawn.cc            |  65 ++---
>  6 files changed, 289 insertions(+), 254 deletions(-)
> 
> -- 
> 2.21.0

Pushed.


Thanks,
Corinna

-- 
Corinna Vinschen
Cygwin Maintainer

Attachment: signature.asc
Description: PGP signature