* fhandler.h (fhandler_pty_master::~fhandler_pty_master): Delete.

* fhandler_tty.cc (fhandler_pty_master::~fhandler_pty_master): Ditto.
This commit is contained in:
Christopher Faylor 2013-12-17 19:05:26 +00:00
parent f51db32d8c
commit 891b1e7e22
4 changed files with 8 additions and 10 deletions

View File

@ -1,3 +1,8 @@
2013-12-17 Christopher Faylor <me.cygwin2013@cgf.cx>
* fhandler.h (fhandler_pty_master::~fhandler_pty_master): Delete.
* fhandler_tty.cc (fhandler_pty_master::~fhandler_pty_master): Ditto.
2013-12-11 Corinna Vinschen <corinna@vinschen.de>
* syscalls.cc (NT_TRANSACTIONAL_ERROR): Define.

View File

@ -1573,7 +1573,6 @@ public:
void flush_to_slave ();
fhandler_pty_master (void *) {}
~fhandler_pty_master ();
void copyto (fhandler_base *x)
{

View File

@ -1326,15 +1326,6 @@ fhandler_pty_master::close ()
return 0;
}
/* This is just to catch error conditions. Since the constructor
actually opens some handles, and stat() does not open an fd, they need
to be closed when the fhandler goes away. */
fhandler_pty_master::~fhandler_pty_master ()
{
if (from_master && to_master)
close_with_arch ();
}
ssize_t __stdcall
fhandler_pty_master::write (const void *ptr, size_t len)
{

View File

@ -13,3 +13,6 @@ Bug Fixes
despite the fact that the filesystem claims to handle them, stop transaction
and try again without.
Fixes: http://cygwin.com/ml/cygwin/2013-12/msg00119.html
- Closing a dup'ed master tty handle should no longer invalidate the dup'ed fd.
Fixes: http://cygwin.com/ml/cygwin/2013-12/threads.html#00279