* fhandler.cc (fhandler_base_overlapped::wait_overlapped): Only raise SIGPIPE

when writing.
* fhandler.h: Include "tty.h".
(fhandler_termios::_tc): Rename from tc.
(fhandler_termios::tc): New method.
(fhandler_termios::tcinit): Remove an argument.
(fhandler_termios::get_ttyp): Use method to retrieve value.
(fhandler_console::console_state): Move here.
(fhandler_console::dev_state): Delete.
(fhandler_console::shared_console_info): Define.
(fhandler_console::open_shared_console): Move this function under
fhandler_console umbrella.
(fhandler_console::tc): Define.  Return static value.
(fhandler_console::focus_aware): Accommodate deletion of dev_state.
(fhandler_console): Add tty_list::get_cttyp as a friend.
* fhandler_console.cc (dev_state): Redefine as a pointer within
shared_console_info and change dev-> to dev.  throughout.
(fhandler_console::shared_console_info): Move into fhandler_console.
(fhandler_console::open_shared_console): Move into fhandler_console change
argument to simple bool.
(enum_windows): Accommodate changes to console_state and open_shared_console.
(console_unit::console_unit): Ditto.
(fhandler_console::get_tty_stuff): Accommodate change to dev_state.
(tty_list::get_cttyp): Accommodate change to
handler_console::shared_console_info.
(fhandler_console::read): Accommodate change from tc to tc ().
(fhandler_console::set_input_state): Ditto.
(fhandler_console::open): Accommodate tcinit argument change and change from tc
to tc().
(fhandler_console::input_tcsetattr): Accomodate change from tc to tc().
(fhandler_console::input_tcsetattr): Ditto.
(fhandler_console::write_normal): Ditto.
(fhandler_console::init): Ditto.
(fhandler_console::igncr_enabled): Ditto.
* fhandler_termios.cc (fhandler_termios::tcinit): Remove first argument.
Expect tc() to have been set up first.  Use tc() rather than tc.
(fhandler_termios::tcsetpgrp): Accomodate change from tc to tc().
(fhandler_termios::tcgetpgrp): Ditto.
(fhandler_termios::bg_check): Ditto.
(fhandler_termios::line_edit: Ditto.
(fhandler_tty_master::set_winsize): Ditto.
(fhandler_tty_slave::open): Ditto.
(fhandler_tty_slave::init): Ditto.
(fhandler_pty_master::write): Ditto.
(fhandler_pty_master::setup): Ditto.  Accommodate change in arguments to
tcinit.
(fhandler_tty_slave::fch_open_handles): Set _tc directly.
(tty_min::is_orphaned_process_group): Don't assume that parent pid exists.
* pinfo.cc (_pinfo::set_ctty): Reset myself->{pgid,sid} here if we were started
by a non-Cygwin process but the tty exists.
* shared_info.h (console_state): Delete from here.
* tty.h: Make multiple inclusion safe.

diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog
index ae356b0..af2230c 100644 (file)
--- a/winsup/cygwin/ChangeLog
+++ b/winsup/cygwin/ChangeLog
@@ -1,3 +1,61 @@
+2011-06-03  Christopher Faylor  <me.cygwin2011@cgf.cx>
+
+       * fhandler.cc (fhandler_base_overlapped::wait_overlapped): Only raise
+       SIGPIPE when writing.
+
+       * fhandler.h: Include "tty.h".
+       (fhandler_termios::_tc): Rename from tc.
+       (fhandler_termios::tc): New method.
+       (fhandler_termios::tcinit): Remove an argument.
+       (fhandler_termios::get_ttyp): Use method to retrieve value.
+       (fhandler_console::console_state): Move here.
+       (fhandler_console::dev_state): Delete.
+       (fhandler_console::shared_console_info): Define.
+       (fhandler_console::open_shared_console): Move this function under
+       fhandler_console umbrella.
+       (fhandler_console::tc): Define.  Return static value.
+       (fhandler_console::focus_aware): Accommodate deletion of dev_state.
+       (fhandler_console): Add tty_list::get_cttyp as a friend.
+       * fhandler_console.cc (dev_state): Redefine as a pointer within
+       shared_console_info and change dev-> to dev. throughout.
+       (fhandler_console::shared_console_info): Move into fhandler_console.
+       (fhandler_console::open_shared_console): Move into fhandler_console
+       change argument to simple bool.
+       (enum_windows): Accommodate changes to console_state and
+       open_shared_console.
+       (console_unit::console_unit): Ditto.
+       (fhandler_console::get_tty_stuff): Accommodate change to dev_state.
+       (tty_list::get_cttyp): Accommodate change to
+       handler_console::shared_console_info.
+       (fhandler_console::read): Accommodate change from tc to tc ().
+       (fhandler_console::set_input_state): Ditto.
+       (fhandler_console::open): Accommodate tcinit argument change and change
+       from tc to tc().
+       (fhandler_console::input_tcsetattr): Accomodate change from tc to tc().
+       (fhandler_console::input_tcsetattr): Ditto.
+       (fhandler_console::write_normal): Ditto.
+       (fhandler_console::init): Ditto.
+       (fhandler_console::igncr_enabled): Ditto.
+       * fhandler_termios.cc (fhandler_termios::tcinit): Remove first argument.
+       Expect tc() to have been set up first.  Use tc() rather than tc.
+       (fhandler_termios::tcsetpgrp): Accomodate change from tc to tc().
+       (fhandler_termios::tcgetpgrp): Ditto.
+       (fhandler_termios::bg_check): Ditto.
+       (fhandler_termios::line_edit: Ditto.
+       (fhandler_tty_master::set_winsize): Ditto.
+       (fhandler_tty_slave::open): Ditto.
+       (fhandler_tty_slave::init): Ditto.
+       (fhandler_pty_master::write): Ditto.
+       (fhandler_pty_master::setup): Ditto.  Accommodate change in arguments
+       to tcinit.
+       (fhandler_tty_slave::fch_open_handles): Set _tc directly.
+       (tty_min::is_orphaned_process_group): Don't assume that parent pid
+       exists.
+       * pinfo.cc (_pinfo::set_ctty): Reset myself->{pgid,sid} here if we were
+       started by a non-Cygwin process but the tty exists.
+       * shared_info.h (console_state): Delete from here.
+       * tty.h: Make multiple inclusion safe.
+
 2011-05-31  Christopher Faylor  <me.cygwin2011@cgf.cx>
 
        * exceptions.cc (ctrl_c_handler): Simplify test for no parent tty.

diff --git a/winsup/cygwin/fhandler.cc b/winsup/cygwin/fhandler.cc
index 1cee133..4ea8497 100644 (file)
--- a/winsup/cygwin/fhandler.cc
+++ b/winsup/cygwin/fhandler.cc
@@ -1883,13 +1883,15 @@ fhandler_base_overlapped::wait_overlapped (bool inres, bool writing, DWORD *byte
       debug_printf ("EOF");
       *bytes = 0;
       res = overlapped_success;
+      if (writing && err == ERROR_BROKEN_PIPE)
+       raise (SIGPIPE);
     }
   else
     {
       debug_printf ("res %u, err %u", (unsigned) res, err);
       *bytes = (DWORD) -1;
       __seterrno_from_win_error (err);
-      if (writing && (err == ERROR_NO_DATA || err == ERROR_BROKEN_PIPE))
+      if (writing && err == ERROR_NO_DATA)
        raise (SIGPIPE);
     }
 

diff --git a/winsup/cygwin/fhandler.h b/winsup/cygwin/fhandler.h
index 2e7f6b5..d62a752 100644 (file)
--- a/winsup/cygwin/fhandler.h
+++ b/winsup/cygwin/fhandler.h
@@ -12,6 +12,7 @@ details. */
 #ifndef _FHANDLER_H_
 #define _FHANDLER_H_
 
+#include "tty.h"
 /* fcntl flags used only internaly. */
 #define O_NOSYMLINK 0x080000
 #define O_DIROPEN   0x100000
@@ -925,7 +926,8 @@ class fhandler_termios: public fhandler_base
   virtual void doecho (const void *, DWORD) {};
   virtual int accept_input () {return 1;};
  public:
-  tty_min *tc;
+  tty_min *_tc;
+  virtual tty_min *tc () const {return _tc; }
   fhandler_termios () :
   fhandler_base ()
   {
@@ -934,9 +936,9 @@ class fhandler_termios: public fhandler_base
   HANDLE& get_output_handle () { return output_handle; }
   line_edit_status line_edit (const char *rptr, int nread, termios&);
   void set_output_handle (HANDLE h) { output_handle = h; }
-  void tcinit (tty_min *this_tc, bool force);
+  void tcinit (bool force);
   bool is_tty () const { return true; }
-  tty *get_ttyp () { return (tty *) tc; }
+  tty *get_ttyp () { return (tty *) tc (); }
   void sigflush ();
   int tcgetpgrp ();
   int tcsetpgrp (int pid);
@@ -1041,9 +1043,15 @@ class dev_console
 /* This is a input and output console handle */
 class fhandler_console: public fhandler_termios
 {
- private:
+public:
+  struct console_state
+  {
+    tty_min tty_min_state;
+    dev_console dev_state;
+  };
+private:
   static const unsigned MAX_WRITE_CHARS;
-  static dev_console *dev_state;
+  static console_state *shared_console_info;
   static bool invisible_console;
 
   /* Used when we encounter a truncated multi-byte sequence.  The
@@ -1075,9 +1083,16 @@ class fhandler_console: public fhandler_termios
   void set_cursor_maybe ();
   static bool create_invisible_console (HWINSTA);
   static bool create_invisible_console_workaround ();
+  static console_state *open_shared_console (HWND, HANDLE&, bool&);
+  tty_min *tc () const {return &(shared_console_info->tty_min_state);}
 
  public:
   fhandler_console ();
+  static console_state *open_shared_console (HWND hw, HANDLE& h)
+  {
+    bool createit = false;
+    return open_shared_console (hw, h, createit);
+  }
 
   fhandler_console* is_console () { return this; }
 
@@ -1098,7 +1113,7 @@ class fhandler_console: public fhandler_termios
   int ioctl (unsigned int cmd, void *);
   int init (HANDLE, DWORD, mode_t);
   bool mouse_aware (MOUSE_EVENT_RECORD& mouse_event);
-  bool focus_aware () {return dev_state->use_focus;}
+  bool focus_aware () {return shared_console_info->dev_state.use_focus;}
 
   select_record *select_read (select_stuff *);
   select_record *select_write (select_stuff *);
@@ -1114,6 +1129,7 @@ class fhandler_console: public fhandler_termios
   static bool need_invisible ();
   static bool has_a () {return !invisible_console;}
   size_t size () const { return sizeof (*this);}
+  friend tty_min * tty_list::get_cttyp ();
 };
 
 class fhandler_tty_common: public fhandler_termios

diff --git a/winsup/cygwin/fhandler_console.cc b/winsup/cygwin/fhandler_console.cc
index 4faa605..399830b 100644 (file)
--- a/winsup/cygwin/fhandler_console.cc
+++ b/winsup/cygwin/fhandler_console.cc
@@ -45,8 +45,9 @@ details. */
  * Negative values represents current screen dimensions
  */
 
-#define srTop (dev_state->info.winTop + dev_state->scroll_region.Top)
-#define srBottom ((dev_state->scroll_region.Bottom < 0) ? dev_state->info.winBottom : dev_state->info.winTop + dev_state->scroll_region.Bottom)
+#define dev_state (shared_console_info->dev_state)
+#define srTop (dev_state.info.winTop + dev_state.scroll_region.Top)
+#define srBottom ((dev_state.scroll_region.Bottom < 0) ? dev_state.info.winBottom : dev_state.info.winTop + dev_state.scroll_region.Bottom)
 
 #define use_tty ISSTATE (myself, PID_USETTY)
 
@@ -54,12 +55,10 @@ const char *get_nonascii_key (INPUT_RECORD&, char *);
 
 const unsigned fhandler_console::MAX_WRITE_CHARS = 16384;
 
-static console_state NO_COPY *shared_console_info;
+fhandler_console::console_state NO_COPY *fhandler_console::shared_console_info;
 
 bool NO_COPY fhandler_console::invisible_console;
 
-dev_console NO_COPY *fhandler_console::dev_state;
-
 static void
 beep ()
 {
@@ -78,15 +77,18 @@ beep ()
   MessageBeep (MB_OK);
 }
 
-console_state *
-open_shared_console (HWND hw, HANDLE& h, shared_locations& m)
+fhandler_console::console_state *
+fhandler_console::open_shared_console (HWND hw, HANDLE& h, bool& create)
 {
   wchar_t namebuf[(sizeof "XXXXXXXXXXXXXXXXXX-consNNNNNNNNNN")];
   __small_swprintf (namebuf, L"%S-cons%p", &installation_key, hw);
   h = NULL;
-  return (console_state *) open_shared (namebuf, 0, h,
-                                       sizeof (*shared_console_info),
-                                       &m); 
+
+  shared_locations m = create ? SH_JUSTCREATE: SH_JUSTOPEN;
+  console_state *res = (console_state *)
+    open_shared (namebuf, 0, h, sizeof (*shared_console_info), &m); 
+  create = m == SH_JUSTCREATE;
+  return res;
 }
 class console_unit
 {
@@ -106,10 +108,9 @@ enum_windows (HWND hw, LPARAM lp)
   console_unit *this1 = (console_unit *) lp;
   if (hw == this1->me)
     return TRUE;
-  shared_locations m = SH_JUSTOPEN;
   HANDLE h;
-  console_state *cs;
-  if ((cs = open_shared_console (hw, h, m)))
+  fhandler_console::console_state *cs;
+  if ((cs = fhandler_console::open_shared_console (hw, h)))
     {
       this1->bitmask ^= 1 << cs->tty_min_state.getntty ();
       UnmapViewOfFile ((void *) cs);
@@ -130,24 +131,23 @@ console_unit::console_unit (HWND me0):
 bool
 fhandler_console::set_unit ()
 {
-  bool need_initializing;
+  bool created;
   if (shared_console_info)
-    need_initializing = false;
+    created = false;
   else
     {
       HWND me = GetConsoleWindow ();
-      shared_locations m = SH_JUSTCREATE;
-      shared_console_info = open_shared_console (me, cygheap->console_h, m);
+      created = true;
+      shared_console_info = open_shared_console (me, cygheap->console_h, created);
       ProtectHandleINH (cygheap->console_h);
-      if ((need_initializing = m != SH_JUSTOPEN))
+      if (created)
        {
          lock_ttys here;
          shared_console_info->tty_min_state.setntty (DEV_CONS_MAJOR, console_unit (me));
        }
     }
 
-  dev_state = &shared_console_info->dev_state;
-  return need_initializing;
+  return created;
 }
 
 /* Allocate and initialize the shared record for the current console.
@@ -158,16 +158,16 @@ fhandler_console::get_tty_stuff ()
   if (set_unit ())
       {
 
-       dev_state->scroll_region.Bottom = -1;
-       dev_state->dwLastCursorPosition.X = -1;
-       dev_state->dwLastCursorPosition.Y = -1;
-       dev_state->dwLastMousePosition.X = -1;
-       dev_state->dwLastMousePosition.Y = -1;
-       dev_state->dwLastButtonState = 0;       /* none pressed */
-       dev_state->last_button_code = 3;        /* released */
-       dev_state->underline_color = FOREGROUND_GREEN | FOREGROUND_BLUE;
-       dev_state->dim_color = FOREGROUND_RED | FOREGROUND_GREEN | FOREGROUND_BLUE;
-       dev_state->meta_mask = LEFT_ALT_PRESSED;
+       dev_state.scroll_region.Bottom = -1;
+       dev_state.dwLastCursorPosition.X = -1;
+       dev_state.dwLastCursorPosition.Y = -1;
+       dev_state.dwLastMousePosition.X = -1;
+       dev_state.dwLastMousePosition.Y = -1;
+       dev_state.dwLastButtonState = 0;        /* none pressed */
+       dev_state.last_button_code = 3; /* released */
+       dev_state.underline_color = FOREGROUND_GREEN | FOREGROUND_BLUE;
+       dev_state.dim_color = FOREGROUND_RED | FOREGROUND_GREEN | FOREGROUND_BLUE;
+       dev_state.meta_mask = LEFT_ALT_PRESSED;
        /* Set the mask that determines if an input keystroke is modified by
           META.  We set this based on the keyboard layout language loaded
           for the current thread.  The left <ALT> key always generates
@@ -178,9 +178,9 @@ fhandler_console::get_tty_stuff ()
           these keyboards right <ALT> (called AltGr) is used to produce the
           shell symbols and should not be interpreted as META. */
        if (PRIMARYLANGID (LOWORD (GetKeyboardLayout (0))) == LANG_ENGLISH)
-         dev_state->meta_mask |= RIGHT_ALT_PRESSED;
-       dev_state->set_default_attr ();
-       dev_state->backspace_keycode = CERASE;
+         dev_state.meta_mask |= RIGHT_ALT_PRESSED;
+       dev_state.set_default_attr ();
+       dev_state.backspace_keycode = CERASE;
        shared_console_info->tty_min_state.sethwnd ((HWND) INVALID_HANDLE_VALUE);
       }
 
@@ -194,7 +194,8 @@ tty_list::get_cttyp ()
 {
   _dev_t n = myself->ctty;
   if (iscons_dev (n))
-    return shared_console_info ? &shared_console_info->tty_min_state : NULL;
+    return fhandler_console::shared_console_info ?
+      &fhandler_console::shared_console_info->tty_min_state : NULL;
   else if (istty_slave_dev (n))
     return &ttys[device::minor (n)];
   else
@@ -225,9 +226,9 @@ dev_console::str_to_con (mbtowc_p f_mbtowc, const char *charset,
 bool
 fhandler_console::set_raw_win32_keyboard_mode (bool new_mode)
 {
-  bool old_mode = dev_state->raw_win32_keyboard_mode;
-  dev_state->raw_win32_keyboard_mode = new_mode;
-  syscall_printf ("raw keyboard mode %sabled", dev_state->raw_win32_keyboard_mode ? "en" : "dis");
+  bool old_mode = dev_state.raw_win32_keyboard_mode;
+  dev_state.raw_win32_keyboard_mode = new_mode;
+  syscall_printf ("raw keyboard mode %sabled", dev_state.raw_win32_keyboard_mode ? "en" : "dis");
   return old_mode;
 };
 
@@ -239,30 +240,30 @@ fhandler_console::set_cursor_maybe ()
   if (!GetConsoleScreenBufferInfo (get_output_handle (), &now))
     return;
 
-  if (dev_state->dwLastCursorPosition.X != now.dwCursorPosition.X ||
-      dev_state->dwLastCursorPosition.Y != now.dwCursorPosition.Y)
+  if (dev_state.dwLastCursorPosition.X != now.dwCursorPosition.X ||
+      dev_state.dwLastCursorPosition.Y != now.dwCursorPosition.Y)
     {
       SetConsoleCursorPosition (get_output_handle (), now.dwCursorPosition);
-      dev_state->dwLastCursorPosition = now.dwCursorPosition;
+      dev_state.dwLastCursorPosition = now.dwCursorPosition;
     }
 }
 
 void
 fhandler_console::send_winch_maybe ()
 {
-  SHORT y = dev_state->info.dwWinSize.Y;
-  SHORT x = dev_state->info.dwWinSize.X;
-  dev_state->fillin_info (get_output_handle ());
+  SHORT y = dev_state.info.dwWinSize.Y;
+  SHORT x = dev_state.info.dwWinSize.X;
+  dev_state.fillin_info (get_output_handle ());
 
-  if (y != dev_state->info.dwWinSize.Y || x != dev_state->info.dwWinSize.X)
+  if (y != dev_state.info.dwWinSize.Y || x != dev_state.info.dwWinSize.X)
     {
       extern fhandler_tty_master *tty_master;
-      dev_state->scroll_region.Top = 0;
-      dev_state->scroll_region.Bottom = -1;
+      dev_state.scroll_region.Top = 0;
+      dev_state.scroll_region.Bottom = -1;
       if (tty_master)
        tty_master->set_winsize (true);
       else
-       tc->kill_pgrp (SIGWINCH);
+       tc ()->kill_pgrp (SIGWINCH);
     }
 }
 
@@ -270,7 +271,7 @@ fhandler_console::send_winch_maybe ()
 bool
 fhandler_console::mouse_aware (MOUSE_EVENT_RECORD& mouse_event)
 {
-  if (! dev_state->use_mouse)
+  if (!dev_state.use_mouse)
     return 0;
 
   /* Adjust mouse position by window scroll buffer offset
@@ -278,8 +279,8 @@ fhandler_console::mouse_aware (MOUSE_EVENT_RECORD& mouse_event)
   CONSOLE_SCREEN_BUFFER_INFO now;
   if (GetConsoleScreenBufferInfo (get_output_handle (), &now))
     {
-      dev_state->dwMousePosition.X = mouse_event.dwMousePosition.X - now.srWindow.Left;
-      dev_state->dwMousePosition.Y = mouse_event.dwMousePosition.Y - now.srWindow.Top;
+      dev_state.dwMousePosition.X = mouse_event.dwMousePosition.X - now.srWindow.Left;
+      dev_state.dwMousePosition.Y = mouse_event.dwMousePosition.Y - now.srWindow.Top;
     }
   else
     {
@@ -288,21 +289,21 @@ fhandler_console::mouse_aware (MOUSE_EVENT_RECORD& mouse_event)
     }
 
   /* Check whether adjusted mouse position can be reported */
-  if (dev_state->dwMousePosition.X > 0xFF - ' ' - 1
-      || dev_state->dwMousePosition.Y > 0xFF - ' ' - 1)
+  if (dev_state.dwMousePosition.X > 0xFF - ' ' - 1
+      || dev_state.dwMousePosition.Y > 0xFF - ' ' - 1)
     {
       /* Mouse position out of reporting range */
       return 0;
     }
 
   return ((mouse_event.dwEventFlags == 0 || mouse_event.dwEventFlags == DOUBLE_CLICK)
-         && mouse_event.dwButtonState != dev_state->dwLastButtonState)
+         && mouse_event.dwButtonState != dev_state.dwLastButtonState)
         || mouse_event.dwEventFlags == MOUSE_WHEELED
         || (mouse_event.dwEventFlags == MOUSE_MOVED
-            && (dev_state->dwMousePosition.X != dev_state->dwLastMousePosition.X
-                || dev_state->dwMousePosition.Y != dev_state->dwLastMousePosition.Y)
-            && ((dev_state->use_mouse >= 2 && mouse_event.dwButtonState)
-                || dev_state->use_mouse >= 3));
+            && (dev_state.dwMousePosition.X != dev_state.dwLastMousePosition.X
+                || dev_state.dwMousePosition.Y != dev_state.dwLastMousePosition.Y)
+            && ((dev_state.use_mouse >= 2 && mouse_event.dwButtonState)
+                || dev_state.use_mouse >= 3));
 }
 
 void __stdcall
@@ -330,7 +331,7 @@ fhandler_console::read (void *pv, size_t& buflen)
   DWORD timeout = is_nonblocking () ? 0 : INFINITE;
   char tmp[60];
 
-  termios ti = tc->ti;
+  termios ti = tc ()->ti;
   for (;;)
     {
       int bgres;
@@ -379,7 +380,7 @@ restart:
 #define virtual_key_code (input_rec.Event.KeyEvent.wVirtualKeyCode)
 #define control_key_state (input_rec.Event.KeyEvent.dwControlKeyState)
 
-         dev_state->nModifiers = 0;
+         dev_state.nModifiers = 0;
 
 #ifdef DEBUGGING
          /* allow manual switching to/from raw mode via ctrl-alt-scrolllock */
@@ -388,12 +389,12 @@ restart:
              ((control_key_state & (LEFT_ALT_PRESSED | LEFT_CTRL_PRESSED)) == (LEFT_ALT_PRESSED | LEFT_CTRL_PRESSED))
            )
            {
-             set_raw_win32_keyboard_mode (!dev_state->raw_win32_keyboard_mode);
+             set_raw_win32_keyboard_mode (!dev_state.raw_win32_keyboard_mode);
              continue;
            }
 #endif
 
-         if (dev_state->raw_win32_keyboard_mode)
+         if (dev_state.raw_win32_keyboard_mode)
            {
              __small_sprintf (tmp, "\033{%u;%u;%u;%u;%u;%luK",
                                    input_rec.Event.KeyEvent.bKeyDown,
@@ -428,20 +429,20 @@ restart:
            continue;
 
          if (control_key_state & SHIFT_PRESSED)
-           dev_state->nModifiers |= 1;
+           dev_state.nModifiers |= 1;
          if (control_key_state & RIGHT_ALT_PRESSED)
-           dev_state->nModifiers |= 2;
+           dev_state.nModifiers |= 2;
          if (control_key_state & CTRL_PRESSED)
-           dev_state->nModifiers |= 4;
+           dev_state.nModifiers |= 4;
          if (control_key_state & LEFT_ALT_PRESSED)
-           dev_state->nModifiers |= 8;
+           dev_state.nModifiers |= 8;
 
          if (input_rec.Event.KeyEvent.wVirtualScanCode == 14)
            {
-             char c = dev_state->backspace_keycode;
+             char c = dev_state.backspace_keycode;
              nread = 0;
              if (control_key_state & ALT_PRESSED) {
-               if (dev_state->metabit)
+               if (dev_state.metabit)
                  c |= 0x80;
                else
                  tmp[nread++] = '\e';
@@ -461,14 +462,14 @@ restart:
              toadd = get_nonascii_key (input_rec, tmp);
              if (!toadd)
                {
-                 dev_state->nModifiers = 0;
+                 dev_state.nModifiers = 0;
                  continue;
                }
              nread = strlen (toadd);
            }
          else
            {
-             nread = dev_state->con_to_str (tmp + 1, 59, wch);
+             nread = dev_state.con_to_str (tmp + 1, 59, wch);
              /* Determine if the keystroke is modified by META.  The tricky
                 part is to distinguish whether the right Alt key should be
                 recognized as Alt, or as AltGr. */
@@ -492,7 +493,7 @@ restart:
                  else
                    toadd = tmp + 1;
                }
-             else if (dev_state->metabit)
+             else if (dev_state.metabit)
                {
                  tmp[1] |= 0x80;
                  toadd = tmp + 1;
@@ -503,7 +504,7 @@ restart:
                  tmp[1] = cyg_tolower (tmp[1]);
                  toadd = tmp;
                  nread++;
-                 dev_state->nModifiers &= ~4;
+                 dev_state.nModifiers &= ~4;
                }
            }
 #undef ich
@@ -558,30 +559,30 @@ restart:
 
                    if (mouse_event.dwEventFlags == MOUSE_MOVED)
                      {
-                       b = dev_state->last_button_code;
+                       b = dev_state.last_button_code;
                      }
-                   else if (mouse_event.dwButtonState < dev_state->dwLastButtonState)
+                   else if (mouse_event.dwButtonState < dev_state.dwLastButtonState)
                      {
                        b = 3;
                        strcpy (sz, "btn up");
                      }
-                   else if ((mouse_event.dwButtonState & 1) != (dev_state->dwLastButtonState & 1))
+                   else if ((mouse_event.dwButtonState & 1) != (dev_state.dwLastButtonState & 1))
                      {
                        b = 0;
                        strcpy (sz, "btn1 down");
                      }
-                   else if ((mouse_event.dwButtonState & 2) != (dev_state->dwLastButtonState & 2))
+                   else if ((mouse_event.dwButtonState & 2) != (dev_state.dwLastButtonState & 2))
                      {
                        b = 2;
                        strcpy (sz, "btn2 down");
                      }
-                   else if ((mouse_event.dwButtonState & 4) != (dev_state->dwLastButtonState & 4))
+                   else if ((mouse_event.dwButtonState & 4) != (dev_state.dwLastButtonState & 4))
                      {
                        b = 1;
                        strcpy (sz, "btn3 down");
                      }
 
-                   dev_state->last_button_code = b;
+                   dev_state.last_button_code = b;
 
                    if (mouse_event.dwEventFlags == MOUSE_MOVED)
                      {
@@ -591,29 +592,29 @@ restart:
                    else
                      {
                        /* Remember the modified button state */
-                       dev_state->dwLastButtonState = mouse_event.dwButtonState;
+                       dev_state.dwLastButtonState = mouse_event.dwButtonState;
                      }
                  }
 
                /* Remember mouse position */
-               dev_state->dwLastMousePosition.X = dev_state->dwMousePosition.X;
-               dev_state->dwLastMousePosition.Y = dev_state->dwMousePosition.Y;
+               dev_state.dwLastMousePosition.X = dev_state.dwMousePosition.X;
+               dev_state.dwLastMousePosition.Y = dev_state.dwMousePosition.Y;
 
                /* Remember the modifiers */
-               dev_state->nModifiers = 0;
+               dev_state.nModifiers = 0;
                if (mouse_event.dwControlKeyState & SHIFT_PRESSED)
-                   dev_state->nModifiers |= 0x4;
+                   dev_state.nModifiers |= 0x4;
                if (mouse_event.dwControlKeyState & (RIGHT_ALT_PRESSED|LEFT_ALT_PRESSED))
-                   dev_state->nModifiers |= 0x8;
+                   dev_state.nModifiers |= 0x8;
                if (mouse_event.dwControlKeyState & (RIGHT_CTRL_PRESSED|LEFT_CTRL_PRESSED))
-                   dev_state->nModifiers |= 0x10;
+                   dev_state.nModifiers |= 0x10;
 
                /* Indicate the modifiers */
-               b |= dev_state->nModifiers;
+               b |= dev_state.nModifiers;
 
                /* We can now create the code. */
-               sprintf (tmp, "\033[M%c%c%c", b + ' ', dev_state->dwMousePosition.X + ' ' + 1, dev_state->dwMousePosition.Y + ' ' + 1);
-               syscall_printf ("mouse: %s at (%d,%d)", sz, dev_state->dwMousePosition.X, dev_state->dwMousePosition.Y);
+               sprintf (tmp, "\033[M%c%c%c", b + ' ', dev_state.dwMousePosition.X + ' ' + 1, dev_state.dwMousePosition.Y + ' ' + 1);
+               syscall_printf ("mouse: %s at (%d,%d)", sz, dev_state.dwMousePosition.X, dev_state.dwMousePosition.Y);
 
                toadd = tmp;
                nread = 6;
@@ -622,7 +623,7 @@ restart:
          break;
 
        case FOCUS_EVENT:
-         if (dev_state->use_focus) {
+         if (dev_state.use_focus) {
            if (input_rec.Event.FocusEvent.bSetFocus)
              sprintf (tmp, "\033[I");
            else
@@ -676,8 +677,8 @@ sig_exit:
 void
 fhandler_console::set_input_state ()
 {
-  if (tc->rstcons ())
-    input_tcsetattr (0, &tc->ti);
+  if (tc ()->rstcons ())
+    input_tcsetattr (0, &tc ()->ti);
 }
 
 bool
@@ -714,30 +715,30 @@ fhandler_console::scroll_screen (int x1, int y1, int x2, int y2, int xn, int yn)
   CHAR_INFO fill;
   COORD dest;
 
-  dev_state->fillin_info (get_output_handle ());
-  sr1.Left = x1 >= 0 ? x1 : dev_state->info.dwWinSize.X - 1;
+  dev_state.fillin_info (get_output_handle ());
+  sr1.Left = x1 >= 0 ? x1 : dev_state.info.dwWinSize.X - 1;
   if (y1 == 0)
-    sr1.Top = dev_state->info.winTop;
+    sr1.Top = dev_state.info.winTop;
   else
-    sr1.Top = y1 > 0 ? y1 : dev_state->info.winBottom;
-  sr1.Right = x2 >= 0 ? x2 : dev_state->info.dwWinSize.X - 1;
+    sr1.Top = y1 > 0 ? y1 : dev_state.info.winBottom;
+  sr1.Right = x2 >= 0 ? x2 : dev_state.info.dwWinSize.X - 1;
   if (y2 == 0)
-    sr1.Bottom = dev_state->info.winTop;
+    sr1.Bottom = dev_state.info.winTop;
   else
-    sr1.Bottom = y2 > 0 ? y2 : dev_state->info.winBottom;
+    sr1.Bottom = y2 > 0 ? y2 : dev_state.info.winBottom;
   sr2.Top = srTop;
   sr2.Left = 0;
   sr2.Bottom = srBottom;
-  sr2.Right = dev_state->info.dwWinSize.X - 1;
+  sr2.Right = dev_state.info.dwWinSize.X - 1;
   if (sr1.Bottom > sr2.Bottom && sr1.Top <= sr2.Bottom)
     sr1.Bottom = sr2.Bottom;
-  dest.X = xn >= 0 ? xn : dev_state->info.dwWinSize.X - 1;
+  dest.X = xn >= 0 ? xn : dev_state.info.dwWinSize.X - 1;
   if (yn == 0)
-    dest.Y = dev_state->info.winTop;
+    dest.Y = dev_state.info.winTop;
   else
-    dest.Y = yn > 0 ? yn : dev_state->info.winBottom;
+    dest.Y = yn > 0 ? yn : dev_state.info.winBottom;
   fill.Char.AsciiChar = ' ';
-  fill.Attributes = dev_state->current_win32_attr;
+  fill.Attributes = dev_state.current_win32_attr;
   ScrollConsoleScreenBuffer (get_output_handle (), &sr1, &sr2, dest, &fill);
 
   /* ScrollConsoleScreenBuffer on Windows 95 is buggy - when scroll distance
@@ -756,7 +757,7 @@ fhandler_console::open (int flags, mode_t)
 {
   HANDLE h;
 
-  tcinit (get_tty_stuff (), false);
+  tcinit (false);
 
   set_io_handle (NULL);
   set_output_handle (NULL);
@@ -784,15 +785,15 @@ fhandler_console::open (int flags, mode_t)
     }
   set_output_handle (h);
 
-  if (dev_state->fillin_info (get_output_handle ()))
+  if (dev_state.fillin_info (get_output_handle ()))
     {
-      dev_state->current_win32_attr = dev_state->info.wAttributes;
-      if (!dev_state->default_color)
-       dev_state->default_color = dev_state->info.wAttributes;
-      dev_state->set_default_attr ();
+      dev_state.current_win32_attr = dev_state.info.wAttributes;
+      if (!dev_state.default_color)
+       dev_state.default_color = dev_state.info.wAttributes;
+      dev_state.set_default_attr ();
     }
 
-  tc->rstcons (false);
+  tc ()->rstcons (false);
   set_open_status ();
 
   DWORD cflags;
@@ -832,13 +833,13 @@ fhandler_console::ioctl (unsigned int cmd, void *buf)
       case TIOCGWINSZ:
        int st;
 
-       st = dev_state->fillin_info (get_output_handle ());
+       st = dev_state.fillin_info (get_output_handle ());
        if (st)
          {
            /* *not* the buffer size, the actual screen size... */
            /* based on Left Top Right Bottom of srWindow */
-           ((struct winsize *) buf)->ws_row = dev_state->info.dwWinSize.Y;
-           ((struct winsize *) buf)->ws_col = dev_state->info.dwWinSize.X;
+           ((struct winsize *) buf)->ws_row = dev_state.info.dwWinSize.Y;
+           ((struct winsize *) buf)->ws_col = dev_state.info.dwWinSize.X;
            syscall_printf ("WINSZ: (row=%d,col=%d)",
                           ((struct winsize *) buf)->ws_row,
                           ((struct winsize *) buf)->ws_col);
@@ -855,13 +856,13 @@ fhandler_console::ioctl (unsigned int cmd, void *buf)
        bg_check (SIGTTOU);
        return 0;
       case KDGKBMETA:
-       *(int *) buf = (dev_state->metabit) ? K_METABIT : K_ESCPREFIX;
+       *(int *) buf = (dev_state.metabit) ? K_METABIT : K_ESCPREFIX;
        return 0;
       case KDSKBMETA:
        if ((int) buf == K_METABIT)
-         dev_state->metabit = TRUE;
+         dev_state.metabit = TRUE;
        else if ((int) buf == K_ESCPREFIX)
-         dev_state->metabit = FALSE;
+         dev_state.metabit = FALSE;
        else
          {
            set_errno (EINVAL);
@@ -871,7 +872,7 @@ fhandler_console::ioctl (unsigned int cmd, void *buf)
       case TIOCLINUX:
        if (*(unsigned char *) buf == 6)
          {
-           *(unsigned char *) buf = (unsigned char) dev_state->nModifiers;
+           *(unsigned char *) buf = (unsigned char) dev_state.nModifiers;
            return 0;
          }
        else
@@ -936,7 +937,8 @@ fhandler_console::input_tcsetattr (int, struct termios const *t)
      available.  We've got ECHO and ICANON, they've
      got ENABLE_ECHO_INPUT and ENABLE_LINE_INPUT. */
 
-  tc->ti = *t;
+  termios_printf ("this %p, tc () %p, t %p", this, tc (), t);
+  tc ()->ti = *t;
 
   if (t->c_lflag & ECHO)
     {
@@ -964,8 +966,8 @@ fhandler_console::input_tcsetattr (int, struct termios const *t)
   if (use_tty)
     {
       flags = 0; // ENABLE_PROCESSED_OUTPUT | ENABLE_WRAP_AT_EOL_OUTPUT;
-      tc->ti.c_iflag = 0;
-      tc->ti.c_lflag = 0;
+      tc ()->ti.c_iflag = 0;
+      tc ()->ti.c_lflag = 0;
     }
 
   flags |= ENABLE_WINDOW_INPUT | ENABLE_MOUSE_INPUT;
@@ -982,7 +984,7 @@ fhandler_console::input_tcsetattr (int, struct termios const *t)
                      res, t, flags, t->c_lflag, t->c_iflag);
     }
 
-  tc->rstcons (false);
+  tc ()->rstcons (false);
   return res;
 }
 
@@ -999,7 +1001,7 @@ int
 fhandler_console::tcgetattr (struct termios *t)
 {
   int res;
-  *t = tc->ti;
+  *t = tc ()->ti;
 
   t->c_cflag |= CS8;
 
@@ -1104,20 +1106,20 @@ fhandler_console::clear_screen (int x1, int y1, int x2, int y2)
   DWORD done;
   int num;
 
-  dev_state->fillin_info (get_output_handle ());
+  dev_state.fillin_info (get_output_handle ());
 
   if (x1 < 0)
-    x1 = dev_state->info.dwWinSize.X - 1;
+    x1 = dev_state.info.dwWinSize.X - 1;
   if (y1 < 0)
-    y1 = dev_state->info.winBottom;
+    y1 = dev_state.info.winBottom;
   if (x2 < 0)
-    x2 = dev_state->info.dwWinSize.X - 1;
+    x2 = dev_state.info.dwWinSize.X - 1;
   if (y2 < 0)
-    y2 = dev_state->info.winBottom;
+    y2 = dev_state.info.winBottom;
 
-  num = abs (y1 - y2) * dev_state->info.dwBufferSize.X + abs (x1 - x2) + 1;
+  num = abs (y1 - y2) * dev_state.info.dwBufferSize.X + abs (x1 - x2) + 1;
 
-  if ((y2 * dev_state->info.dwBufferSize.X + x2) > (y1 * dev_state->info.dwBufferSize.X + x1))
+  if ((y2 * dev_state.info.dwBufferSize.X + x2) > (y1 * dev_state.info.dwBufferSize.X + x1))
     {
       tlc.X = x1;
       tlc.Y = y1;
@@ -1132,7 +1134,7 @@ fhandler_console::clear_screen (int x1, int y1, int x2, int y2)
                               tlc,
                               &done);
   FillConsoleOutputAttribute (get_output_handle (),
-                              dev_state->current_win32_attr,
+                              dev_state.current_win32_attr,
                               num,
                               tlc,
                               &done);
@@ -1143,16 +1145,16 @@ fhandler_console::cursor_set (bool rel_to_top, int x, int y)
 {
   COORD pos;
 
-  dev_state->fillin_info (get_output_handle ());
-  if (y > dev_state->info.winBottom)
-    y = dev_state->info.winBottom;
+  dev_state.fillin_info (get_output_handle ());
+  if (y > dev_state.info.winBottom)
+    y = dev_state.info.winBottom;
   else if (y < 0)
     y = 0;
   else if (rel_to_top)
-    y += dev_state->info.winTop;
+    y += dev_state.info.winTop;
 
-  if (x > dev_state->info.dwWinSize.X)
-    x = dev_state->info.dwWinSize.X - 1;
+  if (x > dev_state.info.dwWinSize.X)
+    x = dev_state.info.dwWinSize.X - 1;
   else if (x < 0)
     x = 0;
 
@@ -1164,18 +1166,18 @@ fhandler_console::cursor_set (bool rel_to_top, int x, int y)
 void
 fhandler_console::cursor_rel (int x, int y)
 {
-  dev_state->fillin_info (get_output_handle ());
-  x += dev_state->info.dwCursorPosition.X;
-  y += dev_state->info.dwCursorPosition.Y;
+  dev_state.fillin_info (get_output_handle ());
+  x += dev_state.info.dwCursorPosition.X;
+  y += dev_state.info.dwCursorPosition.Y;
   cursor_set (false, x, y);
 }
 
 void
 fhandler_console::cursor_get (int *x, int *y)
 {
-  dev_state->fillin_info (get_output_handle ());
-  *y = dev_state->info.dwCursorPosition.Y;
-  *x = dev_state->info.dwCursorPosition.X;
+  dev_state.fillin_info (get_output_handle ());
+  *y = dev_state.info.dwCursorPosition.Y;
+  *x = dev_state.info.dwCursorPosition.X;
 }
 
 /* VT100 line drawing graphics mode maps `abcdefghijklmnopqrstuvwxyz{|}~ to
@@ -1217,9 +1219,9 @@ static wchar_t __vt100_conv [31] = {
 inline
 bool fhandler_console::write_console (PWCHAR buf, DWORD len, DWORD& done)
 {
-  if (dev_state->iso_2022_G1
-       ? dev_state->vt100_graphics_mode_G1
-       : dev_state->vt100_graphics_mode_G0)
+  if (dev_state.iso_2022_G1
+       ? dev_state.vt100_graphics_mode_G1
+       : dev_state.vt100_graphics_mode_G0)
     for (DWORD i = 0; i < len; i ++)
       if (buf[i] >= (unsigned char) '`' && buf[i] <= (unsigned char) '~')
         buf[i] = __vt100_conv[buf[i] - (unsigned char) '`'];
@@ -1299,120 +1301,120 @@ fhandler_console::char_command (char c)
   switch (c)
     {
     case 'm':   /* Set Graphics Rendition */
-       for (int i = 0; i <= dev_state->nargs_; i++)
-        switch (dev_state->args_[i])
+       for (int i = 0; i <= dev_state.nargs_; i++)
+        switch (dev_state.args_[i])
           {
             case 0:    /* normal color */
-              dev_state->set_default_attr ();
+              dev_state.set_default_attr ();
               break;
             case 1:    /* bold */
-              dev_state->intensity = INTENSITY_BOLD;
+              dev_state.intensity = INTENSITY_BOLD;
               break;
             case 2:    /* dim */
-              dev_state->intensity = INTENSITY_DIM;
+              dev_state.intensity = INTENSITY_DIM;
               break;
             case 4:    /* underlined */
-              dev_state->underline = 1;
+              dev_state.underline = 1;
               break;
             case 5:    /* blink mode */
-              dev_state->blink = true;
+              dev_state.blink = true;
               break;
             case 7:    /* reverse */
-              dev_state->reverse = true;
+              dev_state.reverse = true;
               break;
             case 8:    /* invisible */
-              dev_state->intensity = INTENSITY_INVISIBLE;
+              dev_state.intensity = INTENSITY_INVISIBLE;
               break;
             case 10:   /* end alternate charset */
-              dev_state->alternate_charset_active = false;
+              dev_state.alternate_charset_active = false;
               break;
             case 11:   /* start alternate charset */
-              dev_state->alternate_charset_active = true;
+              dev_state.alternate_charset_active = true;
               break;
             case 22:
             case 28:
-              dev_state->intensity = INTENSITY_NORMAL;
+              dev_state.intensity = INTENSITY_NORMAL;
               break;
             case 24:
-              dev_state->underline = false;
+              dev_state.underline = false;
               break;
             case 25:
-              dev_state->blink = false;
+              dev_state.blink = false;
               break;
             case 27:
-              dev_state->reverse = false;
+              dev_state.reverse = false;
               break;
             case 30:           /* BLACK foreground */
-              dev_state->fg = 0;
+              dev_state.fg = 0;
               break;
             case 31:           /* RED foreground */
-              dev_state->fg = FOREGROUND_RED;
+              dev_state.fg = FOREGROUND_RED;
               break;
             case 32:           /* GREEN foreground */
-              dev_state->fg = FOREGROUND_GREEN;
+              dev_state.fg = FOREGROUND_GREEN;
               break;
             case 33:           /* YELLOW foreground */
-              dev_state->fg = FOREGROUND_RED | FOREGROUND_GREEN;
+              dev_state.fg = FOREGROUND_RED | FOREGROUND_GREEN;
               break;
             case 34:           /* BLUE foreground */
-              dev_state->fg = FOREGROUND_BLUE;
+              dev_state.fg = FOREGROUND_BLUE;
               break;
             case 35:           /* MAGENTA foreground */
-              dev_state->fg = FOREGROUND_RED | FOREGROUND_BLUE;
+              dev_state.fg = FOREGROUND_RED | FOREGROUND_BLUE;
               break;
             case 36:           /* CYAN foreground */
-              dev_state->fg = FOREGROUND_BLUE | FOREGROUND_GREEN;
+              dev_state.fg = FOREGROUND_BLUE | FOREGROUND_GREEN;
               break;
             case 37:           /* WHITE foreg */
-              dev_state->fg = FOREGROUND_BLUE | FOREGROUND_GREEN | FOREGROUND_RED;
+              dev_state.fg = FOREGROUND_BLUE | FOREGROUND_GREEN | FOREGROUND_RED;
               break;
             case 39:
-              dev_state->fg = dev_state->default_color & FOREGROUND_ATTR_MASK;
+              dev_state.fg = dev_state.default_color & FOREGROUND_ATTR_MASK;
               break;
             case 40:           /* BLACK background */
-              dev_state->bg = 0;
+              dev_state.bg = 0;
               break;
             case 41:           /* RED background */
-              dev_state->bg = BACKGROUND_RED;
+              dev_state.bg = BACKGROUND_RED;
               break;
             case 42:           /* GREEN background */
-              dev_state->bg = BACKGROUND_GREEN;
+              dev_state.bg = BACKGROUND_GREEN;
               break;
             case 43:           /* YELLOW background */
-              dev_state->bg = BACKGROUND_RED | BACKGROUND_GREEN;
+              dev_state.bg = BACKGROUND_RED | BACKGROUND_GREEN;
               break;
             case 44:           /* BLUE background */
-              dev_state->bg = BACKGROUND_BLUE;
+              dev_state.bg = BACKGROUND_BLUE;
               break;
             case 45:           /* MAGENTA background */
-              dev_state->bg = BACKGROUND_RED | BACKGROUND_BLUE;
+              dev_state.bg = BACKGROUND_RED | BACKGROUND_BLUE;
               break;
             case 46:           /* CYAN background */
-              dev_state->bg = BACKGROUND_BLUE | BACKGROUND_GREEN;
+              dev_state.bg = BACKGROUND_BLUE | BACKGROUND_GREEN;
               break;
             case 47:    /* WHITE background */
-              dev_state->bg = BACKGROUND_BLUE | BACKGROUND_GREEN | BACKGROUND_RED;
+              dev_state.bg = BACKGROUND_BLUE | BACKGROUND_GREEN | BACKGROUND_RED;
               break;
             case 49:
-              dev_state->bg = dev_state->default_color & BACKGROUND_ATTR_MASK;
+              dev_state.bg = dev_state.default_color & BACKGROUND_ATTR_MASK;
               break;
           }
-       dev_state->set_color (get_output_handle ());
+       dev_state.set_color (get_output_handle ());
       break;
     case 'h':
     case 'l':
-      if (!dev_state->saw_question_mark)
+      if (!dev_state.saw_question_mark)
        {
-         switch (dev_state->args_[0])
+         switch (dev_state.args_[0])
            {
            case 4:    /* Insert mode */
-             dev_state->insert_mode = (c == 'h') ? true : false;
-             syscall_printf ("insert mode %sabled", dev_state->insert_mode ? "en" : "dis");
+             dev_state.insert_mode = (c == 'h') ? true : false;
+             syscall_printf ("insert mode %sabled", dev_state.insert_mode ? "en" : "dis");
              break;
            }
          break;
        }
-      switch (dev_state->args_[0])
+      switch (dev_state.args_[0])
        {
        case 47:   /* Save/Restore screen */
          if (c == 'h') /* save */
@@ -1423,16 +1425,16 @@ fhandler_console::char_command (char c)
              if (!GetConsoleScreenBufferInfo (get_output_handle (), &now))
                break;
 
-             dev_state->savebufsiz.X = now.srWindow.Right - now.srWindow.Left + 1;
-             dev_state->savebufsiz.Y = now.srWindow.Bottom - now.srWindow.Top + 1;
+             dev_state.savebufsiz.X = now.srWindow.Right - now.srWindow.Left + 1;
+             dev_state.savebufsiz.Y = now.srWindow.Bottom - now.srWindow.Top + 1;
 
-             if (dev_state->savebuf)
-               cfree (dev_state->savebuf);
-             dev_state->savebuf = (PCHAR_INFO) cmalloc_abort (HEAP_1_BUF, sizeof (CHAR_INFO) *
-                                            dev_state->savebufsiz.X * dev_state->savebufsiz.Y);
+             if (dev_state.savebuf)
+               cfree (dev_state.savebuf);
+             dev_state.savebuf = (PCHAR_INFO) cmalloc_abort (HEAP_1_BUF, sizeof (CHAR_INFO) *
+                                            dev_state.savebufsiz.X * dev_state.savebufsiz.Y);
 
-             ReadConsoleOutputW (get_output_handle (), dev_state->savebuf,
-                                 dev_state->savebufsiz, cob, &now.srWindow);
+             ReadConsoleOutputW (get_output_handle (), dev_state.savebuf,
+                                 dev_state.savebufsiz, cob, &now.srWindow);
            }
          else          /* restore */
            {
@@ -1442,40 +1444,40 @@ fhandler_console::char_command (char c)
              if (!GetConsoleScreenBufferInfo (get_output_handle (), &now))
                break;
 
-             if (!dev_state->savebuf)
+             if (!dev_state.savebuf)
                break;
 
-             WriteConsoleOutputW (get_output_handle (), dev_state->savebuf,
-                                  dev_state->savebufsiz, cob, &now.srWindow);
+             WriteConsoleOutputW (get_output_handle (), dev_state.savebuf,
+                                  dev_state.savebufsiz, cob, &now.srWindow);
 
-             cfree (dev_state->savebuf);
-             dev_state->savebuf = NULL;
-             dev_state->savebufsiz.X = dev_state->savebufsiz.Y = 0;
+             cfree (dev_state.savebuf);
+             dev_state.savebuf = NULL;
+             dev_state.savebufsiz.X = dev_state.savebufsiz.Y = 0;
            }
          break;
 
        case 67: /* DECBKM ("DEC Backarrow Key Mode") */
-         dev_state->backspace_keycode = (c == 'h' ? CTRL('H') : CERASE);
+         dev_state.backspace_keycode = (c == 'h' ? CTRL('H') : CERASE);
          break;
 
        case 1000: /* Mouse tracking */
-         dev_state->use_mouse = (c == 'h') ? 1 : 0;
-         syscall_printf ("mouse support set to mode %d", dev_state->use_mouse);
+         dev_state.use_mouse = (c == 'h') ? 1 : 0;
+         syscall_printf ("mouse support set to mode %d", dev_state.use_mouse);
          break;
 
        case 1002: /* Mouse button event tracking */
-         dev_state->use_mouse = (c == 'h') ? 2 : 0;
-         syscall_printf ("mouse support set to mode %d", dev_state->use_mouse);
+         dev_state.use_mouse = (c == 'h') ? 2 : 0;
+         syscall_printf ("mouse support set to mode %d", dev_state.use_mouse);
          break;
 
        case 1003: /* Mouse any event tracking */
-         dev_state->use_mouse = (c == 'h') ? 3 : 0;
-         syscall_printf ("mouse support set to mode %d", dev_state->use_mouse);
+         dev_state.use_mouse = (c == 'h') ? 3 : 0;
+         syscall_printf ("mouse support set to mode %d", dev_state.use_mouse);
          break;
 
        case 1004: /* Focus in/out event reporting */
-         dev_state->use_focus = (c == 'h') ? true : false;
-         syscall_printf ("focus reporting set to %d", dev_state->use_focus);
+         dev_state.use_focus = (c == 'h') ? true : false;
+         syscall_printf ("focus reporting set to %d", dev_state.use_focus);
          break;
 
        case 2000: /* Raw keyboard mode */
@@ -1483,12 +1485,12 @@ fhandler_console::char_command (char c)
          break;
 
        default: /* Ignore */
-         syscall_printf ("unknown h/l command: %d", dev_state->args_[0]);
+         syscall_printf ("unknown h/l command: %d", dev_state.args_[0]);
          break;
        }
       break;
     case 'J':
-      switch (dev_state->args_[0])
+      switch (dev_state.args_[0])
        {
        case 0:                 /* Clear to end of screen */
          cursor_get (&x, &y);
@@ -1508,19 +1510,19 @@ fhandler_console::char_command (char c)
       break;
 
     case 'A':
-      cursor_rel (0, -(dev_state->args_[0] ? dev_state->args_[0] : 1));
+      cursor_rel (0, -(dev_state.args_[0] ? dev_state.args_[0] : 1));
       break;
     case 'B':
-      cursor_rel (0, dev_state->args_[0] ? dev_state->args_[0] : 1);
+      cursor_rel (0, dev_state.args_[0] ? dev_state.args_[0] : 1);
       break;
     case 'C':
-      cursor_rel (dev_state->args_[0] ? dev_state->args_[0] : 1, 0);
+      cursor_rel (dev_state.args_[0] ? dev_state.args_[0] : 1, 0);
       break;
     case 'D':
-      cursor_rel (-(dev_state->args_[0] ? dev_state->args_[0] : 1),0);
+      cursor_rel (-(dev_state.args_[0] ? dev_state.args_[0] : 1),0);
       break;
     case 'K':
-      switch (dev_state->args_[0])
+      switch (dev_state.args_[0])
        {
          case 0:               /* Clear to end of line */
            cursor_get (&x, &y);
@@ -1540,78 +1542,78 @@ fhandler_console::char_command (char c)
       break;
     case 'H':
     case 'f':
-      cursor_set (true, (dev_state->args_[1] ? dev_state->args_[1] : 1) - 1,
-                       (dev_state->args_[0] ? dev_state->args_[0] : 1) - 1);
+      cursor_set (true, (dev_state.args_[1] ? dev_state.args_[1] : 1) - 1,
+                       (dev_state.args_[0] ? dev_state.args_[0] : 1) - 1);
       break;
     case 'G':   /* hpa - position cursor at column n - 1 */
       cursor_get (&x, &y);
-      cursor_set (false, (dev_state->args_[0] ? dev_state->args_[0] - 1 : 0), y);
+      cursor_set (false, (dev_state.args_[0] ? dev_state.args_[0] - 1 : 0), y);
       break;
     case 'd':   /* vpa - position cursor at line n */
       cursor_get (&x, &y);
-      cursor_set (true, x, (dev_state->args_[0] ? dev_state->args_[0] - 1 : 0));
+      cursor_set (true, x, (dev_state.args_[0] ? dev_state.args_[0] - 1 : 0));
       break;
     case 's':   /* Save cursor position */
-      cursor_get (&dev_state->savex, &dev_state->savey);
-      dev_state->savey -= dev_state->info.winTop;
+      cursor_get (&dev_state.savex, &dev_state.savey);
+      dev_state.savey -= dev_state.info.winTop;
       break;
     case 'u':   /* Restore cursor position */
-      cursor_set (true, dev_state->savex, dev_state->savey);
+      cursor_set (true, dev_state.savex, dev_state.savey);
       break;
     case 'I':  /* TAB */
       cursor_get (&x, &y);
       cursor_set (false, 8 * (x / 8 + 1), y);
       break;
     case 'L':                          /* AL - insert blank lines */
-      dev_state->args_[0] = dev_state->args_[0] ? dev_state->args_[0] : 1;
+      dev_state.args_[0] = dev_state.args_[0] ? dev_state.args_[0] : 1;
       cursor_get (&x, &y);
-      scroll_screen (0, y, -1, -1, 0, y + dev_state->args_[0]);
+      scroll_screen (0, y, -1, -1, 0, y + dev_state.args_[0]);
       break;
     case 'M':                          /* DL - delete lines */
-      dev_state->args_[0] = dev_state->args_[0] ? dev_state->args_[0] : 1;
+      dev_state.args_[0] = dev_state.args_[0] ? dev_state.args_[0] : 1;
       cursor_get (&x, &y);
-      scroll_screen (0, y + dev_state->args_[0], -1, -1, 0, y);
+      scroll_screen (0, y + dev_state.args_[0], -1, -1, 0, y);
       break;
     case '@':                          /* IC - insert chars */
-      dev_state->args_[0] = dev_state->args_[0] ? dev_state->args_[0] : 1;
+      dev_state.args_[0] = dev_state.args_[0] ? dev_state.args_[0] : 1;
       cursor_get (&x, &y);
-      scroll_screen (x, y, -1, y, x + dev_state->args_[0], y);
+      scroll_screen (x, y, -1, y, x + dev_state.args_[0], y);
       break;
     case 'P':                          /* DC - delete chars */
-      dev_state->args_[0] = dev_state->args_[0] ? dev_state->args_[0] : 1;
+      dev_state.args_[0] = dev_state.args_[0] ? dev_state.args_[0] : 1;
       cursor_get (&x, &y);
-      scroll_screen (x + dev_state->args_[0], y, -1, y, x, y);
+      scroll_screen (x + dev_state.args_[0], y, -1, y, x, y);
       break;
     case 'S':                          /* SF - Scroll forward */
-      dev_state->args_[0] = dev_state->args_[0] ? dev_state->args_[0] : 1;
-      scroll_screen (0, dev_state->args_[0], -1, -1, 0, 0);
+      dev_state.args_[0] = dev_state.args_[0] ? dev_state.args_[0] : 1;
+      scroll_screen (0, dev_state.args_[0], -1, -1, 0, 0);
       break;
     case 'T':                          /* SR - Scroll down */
-      dev_state->fillin_info (get_output_handle ());
-      dev_state->args_[0] = dev_state->args_[0] ? dev_state->args_[0] : 1;
-      scroll_screen (0, 0, -1, -1, 0, dev_state->info.winTop + dev_state->args_[0]);
+      dev_state.fillin_info (get_output_handle ());
+      dev_state.args_[0] = dev_state.args_[0] ? dev_state.args_[0] : 1;
+      scroll_screen (0, 0, -1, -1, 0, dev_state.info.winTop + dev_state.args_[0]);
       break;
     case 'X':                          /* ec - erase chars */
-      dev_state->args_[0] = dev_state->args_[0] ? dev_state->args_[0] : 1;
+      dev_state.args_[0] = dev_state.args_[0] ? dev_state.args_[0] : 1;
       cursor_get (&x, &y);
-      scroll_screen (x + dev_state->args_[0], y, -1, y, x, y);
-      scroll_screen (x, y, -1, y, x + dev_state->args_[0], y);
+      scroll_screen (x + dev_state.args_[0], y, -1, y, x, y);
+      scroll_screen (x, y, -1, y, x + dev_state.args_[0], y);
       break;
     case 'Z':                          /* Back tab */
       cursor_get (&x, &y);
       cursor_set (false, ((8 * (x / 8 + 1)) - 8), y);
       break;
     case 'b':                          /* Repeat char #1 #2 times */
-      if (dev_state->insert_mode)
+      if (dev_state.insert_mode)
        {
          cursor_get (&x, &y);
-         scroll_screen (x, y, -1, y, x + dev_state->args_[1], y);
+         scroll_screen (x, y, -1, y, x + dev_state.args_[1], y);
        }
-      while (dev_state->args_[1]--)
-       WriteFile (get_output_handle (), &dev_state->args_[0], 1, (DWORD *) &x, 0);
+      while (dev_state.args_[1]--)
+       WriteFile (get_output_handle (), &dev_state.args_[0], 1, (DWORD *) &x, 0);
       break;
     case 'c':                          /* u9 - Terminal enquire string */
-      if (dev_state->saw_greater_than_sign)
+      if (dev_state.saw_greater_than_sign)
        /* Generate Secondary Device Attribute report, using 67 = ASCII 'C' 
           to indicate Cygwin (convention used by Rxvt, Urxvt, Screen, Mintty), 
           and cygwin version for terminal version. */
@@ -1624,12 +1626,12 @@ fhandler_console::char_command (char c)
       puts_readahead (buf);
       break;
     case 'n':
-      switch (dev_state->args_[0])
+      switch (dev_state.args_[0])
        {
        case 6:                         /* u7 - Cursor position request */
          cursor_get (&x, &y);
-         y -= dev_state->info.winTop;
-         /* x -= dev_state->info.winLeft;              // not available yet */
+         y -= dev_state.info.winTop;
+         /* x -= dev_state.info.winLeft;               // not available yet */
          __small_sprintf (buf, "\033[%d;%dR", y + 1, x + 1);
          puts_readahead (buf);
          break;
@@ -1638,8 +1640,8 @@ fhandler_console::char_command (char c)
        }
       break;
     case 'r':                          /* Set Scroll region */
-      dev_state->scroll_region.Top = dev_state->args_[0] ? dev_state->args_[0] - 1 : 0;
-      dev_state->scroll_region.Bottom = dev_state->args_[1] ? dev_state->args_[1] - 1 : -1;
+      dev_state.scroll_region.Top = dev_state.args_[0] ? dev_state.args_[0] - 1 : 0;
+      dev_state.scroll_region.Bottom = dev_state.args_[1] ? dev_state.args_[1] - 1 : -1;
       cursor_set (true, 0, 0);
       break;
     case 'g':                          /* TAB set/clear */
@@ -1671,7 +1673,7 @@ fhandler_console::write_normal (const unsigned char *src,
   const unsigned char *found = src;
   size_t ret;
   mbstate_t ps;
-  UINT cp = dev_state->get_console_cp ();
+  UINT cp = dev_state.get_console_cp ();
   const char *charset;
   mbtowc_p f_mbtowc;
 
@@ -1721,7 +1723,7 @@ fhandler_console::write_normal (const unsigned char *src,
       /* Valid multibyte sequence?  Process. */
       if (nfound)
        {
-         buf_len = dev_state->str_to_con (f_mbtowc, charset, write_buf,
+         buf_len = dev_state.str_to_con (f_mbtowc, charset, write_buf,
                                           (const char *) trunc_buf.buf,
                                           nfound - trunc_buf.buf);
          if (!write_console (write_buf, buf_len, done))
@@ -1767,7 +1769,7 @@ do_print:
   if (found != src)
     {
       DWORD len = found - src;
-      buf_len = dev_state->str_to_con (f_mbtowc, charset, write_buf,
+      buf_len = dev_state.str_to_con (f_mbtowc, charset, write_buf,
                                       (const char *) src, len);
       if (!buf_len)
        {
@@ -1777,7 +1779,7 @@ do_print:
          return 0;
        }
 
-      if (dev_state->insert_mode)
+      if (dev_state.insert_mode)
        {
          int x, y;
          cursor_get (&x, &y);
@@ -1802,22 +1804,22 @@ do_print:
       switch (base_chars[*found])
        {
        case SO:        /* Shift Out: Invoke G1 character set (ISO 2022) */
-         dev_state->iso_2022_G1 = true;
+         dev_state.iso_2022_G1 = true;
          break;
        case SI:        /* Shift In: Invoke G0 character set (ISO 2022) */
-         dev_state->iso_2022_G1 = false;
+         dev_state.iso_2022_G1 = false;
          break;
        case BEL:
          beep ();
          break;
        case ESC:
-         dev_state->state_ = gotesc;
+         dev_state.state_ = gotesc;
          break;
        case DWN:
          cursor_get (&x, &y);
          if (y >= srBottom)
            {
-             if (y >= dev_state->info.winBottom && !dev_state->scroll_region.Top)
+             if (y >= dev_state.info.winBottom && !dev_state.scroll_region.Top)
                WriteConsoleW (get_output_handle (), L"\n", 1, &done, 0);
              else
                {
@@ -1825,7 +1827,7 @@ do_print:
                  y--;
                }
            }
-         cursor_set (false, ((tc->ti.c_oflag & ONLCR) ? 0 : x), y + 1);
+         cursor_set (false, ((tc ()->ti.c_oflag & ONLCR) ? 0 : x), y + 1);
          break;
        case BAK:
          cursor_rel (-1, 0);
@@ -1894,8 +1896,8 @@ fhandler_console::write (const void *vsrc, size_t len)
   while (src < end)
     {
       debug_printf ("at %d(%c) state is %d", *src, isprint (*src) ? *src : ' ',
-                   dev_state->state_);
-      switch (dev_state->state_)
+                   dev_state.state_);
+      switch (dev_state.state_)
        {
        case normal:
          src = write_normal (src, end);
@@ -1905,111 +1907,111 @@ fhandler_console::write (const void *vsrc, size_t len)
        case gotesc:
          if (*src == '[')              /* CSI Control Sequence Introducer */
            {
-             dev_state->state_ = gotsquare;
-             dev_state->saw_question_mark = false;
-             dev_state->saw_greater_than_sign = false;
-             for (dev_state->nargs_ = 0; dev_state->nargs_ < MAXARGS; dev_state->nargs_++)
-               dev_state->args_[dev_state->nargs_] = 0;
-             dev_state->nargs_ = 0;
+             dev_state.state_ = gotsquare;
+             dev_state.saw_question_mark = false;
+             dev_state.saw_greater_than_sign = false;
+             for (dev_state.nargs_ = 0; dev_state.nargs_ < MAXARGS; dev_state.nargs_++)
+               dev_state.args_[dev_state.nargs_] = 0;
+             dev_state.nargs_ = 0;
            }
          else if (*src == ']')         /* OSC Operating System Command */
            {
-             dev_state->rarg = 0;
-             dev_state->my_title_buf[0] = '\0';
-             dev_state->state_ = gotrsquare;
+             dev_state.rarg = 0;
+             dev_state.my_title_buf[0] = '\0';
+             dev_state.state_ = gotrsquare;
            }
          else if (*src == '(')         /* Designate G0 character set */
            {
-             dev_state->state_ = gotparen;
+             dev_state.state_ = gotparen;
            }
          else if (*src == ')')         /* Designate G1 character set */
            {
-             dev_state->state_ = gotrparen;
+             dev_state.state_ = gotrparen;
            }
          else if (*src == 'M')         /* Reverse Index (scroll down) */
            {
-             dev_state->fillin_info (get_output_handle ());
-             scroll_screen (0, 0, -1, -1, 0, dev_state->info.winTop + 1);
-             dev_state->state_ = normal;
+             dev_state.fillin_info (get_output_handle ());
+             scroll_screen (0, 0, -1, -1, 0, dev_state.info.winTop + 1);
+             dev_state.state_ = normal;
            }
          else if (*src == 'c')         /* RIS Full Reset */
            {
-             dev_state->set_default_attr ();
-             dev_state->vt100_graphics_mode_G0 = false;
-             dev_state->vt100_graphics_mode_G1 = false;
-             dev_state->iso_2022_G1 = false;
+             dev_state.set_default_attr ();
+             dev_state.vt100_graphics_mode_G0 = false;
+             dev_state.vt100_graphics_mode_G1 = false;
+             dev_state.iso_2022_G1 = false;
              clear_screen (0, 0, -1, -1);
              cursor_set (true, 0, 0);
-             dev_state->state_ = normal;
+             dev_state.state_ = normal;
            }
          else if (*src == '8')         /* DECRC Restore cursor position */
            {
-             cursor_set (true, dev_state->savex, dev_state->savey);
-             dev_state->state_ = normal;
+             cursor_set (true, dev_state.savex, dev_state.savey);
+             dev_state.state_ = normal;
            }
          else if (*src == '7')         /* DECSC Save cursor position */
            {
-             cursor_get (&dev_state->savex, &dev_state->savey);
-             dev_state->savey -= dev_state->info.winTop;
-             dev_state->state_ = normal;
+             cursor_get (&dev_state.savex, &dev_state.savey);
+             dev_state.savey -= dev_state.info.winTop;
+             dev_state.state_ = normal;
            }
          else if (*src == 'R')         /* ? */
-             dev_state->state_ = normal;
+             dev_state.state_ = normal;
          else
            {
-             dev_state->state_ = normal;
+             dev_state.state_ = normal;
            }
          src++;
          break;
        case gotarg1:
          if (isdigit (*src))
            {
-             dev_state->args_[dev_state->nargs_] = dev_state->args_[dev_state->nargs_] * 10 + *src - '0';
+             dev_state.args_[dev_state.nargs_] = dev_state.args_[dev_state.nargs_] * 10 + *src - '0';
              src++;
            }
          else if (*src == ';')
            {
              src++;
-             dev_state->nargs_++;
-             if (dev_state->nargs_ >= MAXARGS)
-               dev_state->nargs_--;
+             dev_state.nargs_++;
+             if (dev_state.nargs_ >= MAXARGS)
+               dev_state.nargs_--;
            }
          else
            {
-             dev_state->state_ = gotcommand;
+             dev_state.state_ = gotcommand;
            }
          break;
        case gotcommand:
          char_command (*src++);
-         dev_state->state_ = normal;
+         dev_state.state_ = normal;
          break;
        case gotrsquare:
          if (isdigit (*src))
-           dev_state->rarg = dev_state->rarg * 10 + (*src - '0');
-         else if (*src == ';' && (dev_state->rarg == 2 || dev_state->rarg == 0))
-           dev_state->state_ = gettitle;
+           dev_state.rarg = dev_state.rarg * 10 + (*src - '0');
+         else if (*src == ';' && (dev_state.rarg == 2 || dev_state.rarg == 0))
+           dev_state.state_ = gettitle;
          else
-           dev_state->state_ = eattitle;
+           dev_state.state_ = eattitle;
          src++;
          break;
        case eattitle:
        case gettitle:
          {
-           int n = strlen (dev_state->my_title_buf);
+           int n = strlen (dev_state.my_title_buf);
            if (*src < ' ')
              {
-               if (*src == '\007' && dev_state->state_ == gettitle)
+               if (*src == '\007' && dev_state.state_ == gettitle)
                  {
                    if (old_title)
-                     strcpy (old_title, dev_state->my_title_buf);
-                   set_console_title (dev_state->my_title_buf);
+                     strcpy (old_title, dev_state.my_title_buf);
+                   set_console_title (dev_state.my_title_buf);
                  }
-               dev_state->state_ = normal;
+               dev_state.state_ = normal;
              }
            else if (n < TITLESIZE)
              {
-               dev_state->my_title_buf[n++] = *src;
-               dev_state->my_title_buf[n] = '\0';
+               dev_state.my_title_buf[n++] = *src;
+               dev_state.my_title_buf[n] = '\0';
              }
            src++;
            break;
@@ -2017,38 +2019,38 @@ fhandler_console::write (const void *vsrc, size_t len)
        case gotsquare:
          if (*src == ';')
            {
-             dev_state->state_ = gotarg1;
-             dev_state->nargs_++;
+             dev_state.state_ = gotarg1;
+             dev_state.nargs_++;
              src++;
            }
          else if (isalpha (*src))
-           dev_state->state_ = gotcommand;
+           dev_state.state_ = gotcommand;
          else if (*src != '@' && !isalpha (*src) && !isdigit (*src))
            {
              if (*src == '?')
-               dev_state->saw_question_mark = true;
+               dev_state.saw_question_mark = true;
              else if (*src == '>')
-               dev_state->saw_greater_than_sign = true;
+               dev_state.saw_greater_than_sign = true;
              /* ignore any extra chars between [ and first arg or command */
              src++;
            }
          else
-           dev_state->state_ = gotarg1;
+           dev_state.state_ = gotarg1;
          break;
        case gotparen:  /* Designate G0 Character Set (ISO 2022) */
          if (*src == '0')
-           dev_state->vt100_graphics_mode_G0 = true;
+           dev_state.vt100_graphics_mode_G0 = true;
          else
-           dev_state->vt100_graphics_mode_G0 = false;
-         dev_state->state_ = normal;
+           dev_state.vt100_graphics_mode_G0 = false;
+         dev_state.state_ = normal;
          src++;
          break;
        case gotrparen: /* Designate G1 Character Set (ISO 2022) */
          if (*src == '0')
-           dev_state->vt100_graphics_mode_G1 = true;
+           dev_state.vt100_graphics_mode_G1 = true;
          else
-           dev_state->vt100_graphics_mode_G1 = false;
-         dev_state->state_ = normal;
+           dev_state.vt100_graphics_mode_G1 = false;
+         dev_state.state_ = normal;
          src++;
          break;
        }
@@ -2155,13 +2157,13 @@ fhandler_console::init (HANDLE h, DWORD a, mode_t bin)
   if (h && h != INVALID_HANDLE_VALUE)
     CloseHandle (h);   /* Reopened by open */
 
-  return !tcsetattr (0, &tc->ti);
+  return !tcsetattr (0, &tc ()->ti);
 }
 
 int
 fhandler_console::igncr_enabled ()
 {
-  return tc->ti.c_iflag & IGNCR;
+  return tc ()->ti.c_iflag & IGNCR;
 }
 
 void
@@ -2183,7 +2185,7 @@ set_console_title (char *title)
 void
 fhandler_console::fixup_after_fork_exec (bool execing)
 {
-  ((fhandler_console *)archetype)->tc = tc = get_tty_stuff ();
+  set_unit ();
 }
 
 // #define WINSTA_ACCESS (WINSTA_READATTRIBUTES | STANDARD_RIGHTS_READ | STANDARD_RIGHTS_WRITE | WINSTA_CREATEDESKTOP | WINSTA_EXITWINDOWS)

diff --git a/winsup/cygwin/fhandler_termios.cc b/winsup/cygwin/fhandler_termios.cc
index b3eb465..8282146 100644 (file)
--- a/winsup/cygwin/fhandler_termios.cc
+++ b/winsup/cygwin/fhandler_termios.cc
@@ -24,49 +24,47 @@ details. */
 /* Common functions shared by tty/console */
 
 void
-fhandler_termios::tcinit (tty_min *this_tc, bool is_pty_master)
+fhandler_termios::tcinit (bool is_pty_master)
 {
   /* Initial termios values */
 
-  tc = this_tc;
-
-  if (is_pty_master || !tc->initialized ())
+  if (is_pty_master || !tc ()->initialized ())
     {
-      tc->ti.c_iflag = BRKINT | ICRNL | IXON;
-      tc->ti.c_oflag = OPOST | ONLCR;
-      tc->ti.c_cflag = B38400 | CS8 | CREAD;
-      tc->ti.c_lflag = ISIG | ICANON | ECHO | IEXTEN;
-
-      tc->ti.c_cc[VDISCARD]    = CFLUSH;
-      tc->ti.c_cc[VEOL]                = CEOL;
-      tc->ti.c_cc[VEOL2]       = CEOL2;
-      tc->ti.c_cc[VEOF]                = CEOF;
-      tc->ti.c_cc[VERASE]      = CERASE;
-      tc->ti.c_cc[VINTR]       = CINTR;
-      tc->ti.c_cc[VKILL]       = CKILL;
-      tc->ti.c_cc[VLNEXT]      = CLNEXT;
-      tc->ti.c_cc[VMIN]                = 1;
-      tc->ti.c_cc[VQUIT]       = CQUIT;
-      tc->ti.c_cc[VREPRINT]    = CRPRNT;
-      tc->ti.c_cc[VSTART]      = CSTART;
-      tc->ti.c_cc[VSTOP]       = CSTOP;
-      tc->ti.c_cc[VSUSP]       = CSUSP;
-      tc->ti.c_cc[VSWTC]       = CSWTCH;
-      tc->ti.c_cc[VTIME]       = 0;
-      tc->ti.c_cc[VWERASE]     = CWERASE;
-
-      tc->ti.c_ispeed = tc->ti.c_ospeed = B38400;
-      tc->pgid = is_pty_master ? 0 : myself->pgid;
-      tc->initialized (true);
+      tc ()->ti.c_iflag = BRKINT | ICRNL | IXON;
+      tc ()->ti.c_oflag = OPOST | ONLCR;
+      tc ()->ti.c_cflag = B38400 | CS8 | CREAD;
+      tc ()->ti.c_lflag = ISIG | ICANON | ECHO | IEXTEN;
+
+      tc ()->ti.c_cc[VDISCARD] = CFLUSH;
+      tc ()->ti.c_cc[VEOL]             = CEOL;
+      tc ()->ti.c_cc[VEOL2]    = CEOL2;
+      tc ()->ti.c_cc[VEOF]             = CEOF;
+      tc ()->ti.c_cc[VERASE]   = CERASE;
+      tc ()->ti.c_cc[VINTR]    = CINTR;
+      tc ()->ti.c_cc[VKILL]    = CKILL;
+      tc ()->ti.c_cc[VLNEXT]   = CLNEXT;
+      tc ()->ti.c_cc[VMIN]             = 1;
+      tc ()->ti.c_cc[VQUIT]    = CQUIT;
+      tc ()->ti.c_cc[VREPRINT] = CRPRNT;
+      tc ()->ti.c_cc[VSTART]   = CSTART;
+      tc ()->ti.c_cc[VSTOP]    = CSTOP;
+      tc ()->ti.c_cc[VSUSP]    = CSUSP;
+      tc ()->ti.c_cc[VSWTC]    = CSWTCH;
+      tc ()->ti.c_cc[VTIME]    = 0;
+      tc ()->ti.c_cc[VWERASE]  = CWERASE;
+
+      tc ()->ti.c_ispeed = tc ()->ti.c_ospeed = B38400;
+      tc ()->pgid = is_pty_master ? 0 : myself->pgid;
+      tc ()->initialized (true);
     }
 }
 
 int
 fhandler_termios::tcsetpgrp (const pid_t pgid)
 {
-  termios_printf ("tty %d pgid %d, sid %d, tsid %d", tc->ntty, pgid,
-                   myself->sid, tc->getsid ());
-  if (myself->sid != tc->getsid ())
+  termios_printf ("tty %d pgid %d, sid %d, tsid %d", tc ()->ntty, pgid,
+                   myself->sid, tc ()->getsid ());
+  if (myself->sid != tc ()->getsid ())
     {
       set_errno (EPERM);
       return -1;
@@ -79,8 +77,8 @@ fhandler_termios::tcsetpgrp (const pid_t pgid)
       switch (res)
        {
        case bg_ok:
-         tc->setpgid (pgid);
-         init_console_handler (tc->gethwnd ());
+         tc ()->setpgid (pgid);
+         init_console_handler (tc ()->gethwnd ());
          res = 0;
          break;
        case bg_signalled:
@@ -100,8 +98,8 @@ fhandler_termios::tcsetpgrp (const pid_t pgid)
 int
 fhandler_termios::tcgetpgrp ()
 {
-  if (myself->ctty != -1 && myself->ctty == tc->ntty)
-    return tc->pgid;
+  if (myself->ctty != -1 && myself->ctty == tc ()->ntty)
+    return tc ()->pgid;
   set_errno (ENOTTY);
   return -1;
 }
@@ -109,27 +107,7 @@ fhandler_termios::tcgetpgrp ()
 int
 fhandler_pty_master::tcgetpgrp ()
 {
-  return tc->pgid;
-}
-
-int
-tty_min::is_orphaned_process_group (int pgid)
-{
-  /* An orphaned process group is a process group in which the parent
-     of every member is either itself a member of the group or is not
-     a member of the group's session. */
-  winpids pids ((DWORD) PID_MAP_RW);
-  for (unsigned i = 0; i < pids.npids; i++)
-    {
-      _pinfo *p = pids[i];
-      if (!p->exists () || p->pgid != pgid)
-        continue;
-      pinfo ppid (p->ppid);
-      if (ppid->pgid != pgid &&
-          ppid->sid == myself->sid)
-        return 0;
-    }
-  return 1;
+  return tc ()->pgid;
 }
 
 void
@@ -154,21 +132,45 @@ tty_min::kill_pgrp (int sig)
     sig_send (myself, si);
 }
 
+int
+tty_min::is_orphaned_process_group (int pgid)
+{
+  /* An orphaned process group is a process group in which the parent
+     of every member is either itself a member of the group or is not
+     a member of the group's session. */
+  termios_printf ("checking pgid %d, my sid %d, my parent %d", pgid, myself->sid, myself->ppid);
+  winpids pids ((DWORD) 0);
+  for (unsigned i = 0; i < pids.npids; i++)
+    {
+      _pinfo *p = pids[i];
+      termios_printf ("checking pid %d - has pgid %d\n", p->pid, p->pgid);
+      if (!p || !p->exists () || p->pgid != pgid)
+        continue;
+      pinfo ppid (p->ppid);
+      if (!ppid)
+       continue;
+      termios_printf ("ppid->pgid %d, ppid->sid %d", ppid->pgid, ppid->sid);
+      if (ppid->pgid != pgid && ppid->sid == myself->sid)
+        return 0;
+    }
+  return 1;
+}
+
 bg_check_types
 fhandler_termios::bg_check (int sig)
 {
-  if (!myself->pgid || tc->getpgid () == myself->pgid ||
-       myself->ctty != tc->ntty ||
-       ((sig == SIGTTOU) && !(tc->ti.c_lflag & TOSTOP)))
+  if (!myself->pgid || tc ()->getpgid () == myself->pgid ||
+       myself->ctty != tc ()->ntty ||
+       ((sig == SIGTTOU) && !(tc ()->ti.c_lflag & TOSTOP)))
     return bg_ok;
 
   if (sig < 0)
     sig = -sig;
 
-  termios_printf ("bg I/O pgid %d, tpgid %d, %s, ntty %s", myself->pgid, tc->getpgid (),
-                 myctty (), tc->ttyname ());
+  termios_printf ("bg I/O pgid %d, tpgid %d, %s, ntty %s", myself->pgid, tc ()->getpgid (),
+                 myctty (), tc ()->ttyname ());
 
-  if (tc->getsid () == 0)
+  if (tc ()->getsid () == 0)
     {
       /* The pty has been closed by the master.  Return an EOF
         indication.  FIXME: There is nothing to stop somebody
@@ -188,7 +190,7 @@ fhandler_termios::bg_check (int sig)
      process is orphaned, in which case we return EIO). */
   if (sigs_ignored)
     return bg_ok;   /* Just allow the IO */
-  else if ( tc->is_orphaned_process_group (myself->pgid) )
+  else if (tc ()->is_orphaned_process_group (myself->pgid))
     {
       termios_printf ("process group is orphaned");
       set_errno (EIO);   /* This is an IO error */
@@ -214,7 +216,7 @@ fhandler_termios::bg_check (int sig)
 inline void
 fhandler_termios::echo_erase (int force)
 {
-  if (force || tc->ti.c_lflag & ECHO)
+  if (force || tc ()->ti.c_lflag & ECHO)
     doecho ("\b \b", 3);
 }
 
@@ -269,7 +271,7 @@ fhandler_termios::line_edit (const char *rptr, int nread, termios& ti)
 
          termios_printf ("got interrupt %d, sending signal %d", c, sig);
          eat_readahead (-1);
-         tc->kill_pgrp (sig);
+         tc ()->kill_pgrp (sig);
          ti.c_lflag &= ~FLUSHO;
          sawsig = true;
          goto restart_output;
@@ -279,9 +281,9 @@ fhandler_termios::line_edit (const char *rptr, int nread, termios& ti)
        {
          if (CCEQ (ti.c_cc[VSTOP], c))
            {
-             if (!tc->output_stopped)
+             if (!tc ()->output_stopped)
                {
-                 tc->output_stopped = 1;
+                 tc ()->output_stopped = 1;
                  acquire_output_mutex (INFINITE);
                }
              continue;
@@ -289,11 +291,11 @@ fhandler_termios::line_edit (const char *rptr, int nread, termios& ti)
          else if (CCEQ (ti.c_cc[VSTART], c))
            {
     restart_output:
-             tc->output_stopped = 0;
+             tc ()->output_stopped = 0;
              release_output_mutex ();
              continue;
            }
-         else if ((ti.c_iflag & IXANY) && tc->output_stopped)
+         else if ((ti.c_iflag & IXANY) && tc ()->output_stopped)
            goto restart_output;
        }
       if (iscanon && ti.c_lflag & IEXTEN && CCEQ (ti.c_cc[VDISCARD], c))

diff --git a/winsup/cygwin/fhandler_tty.cc b/winsup/cygwin/fhandler_tty.cc
index 2393f70..32a957a 100644 (file)
--- a/winsup/cygwin/fhandler_tty.cc
+++ b/winsup/cygwin/fhandler_tty.cc
@@ -68,7 +68,7 @@ fhandler_tty_master::set_winsize (bool sendSIGWINCH)
   console->ioctl (TIOCGWINSZ, &w);
   get_ttyp ()->winsize = w;
   if (sendSIGWINCH)
-    tc->kill_pgrp (SIGWINCH);
+    tc ()->kill_pgrp (SIGWINCH);
 }
 
 int
@@ -493,7 +493,9 @@ fhandler_tty_slave::open (int flags, mode_t)
   for (HANDLE **h = handles; *h; h++)
     **h = NULL;
 
-  tcinit (cygwin_shared->tty[get_unit ()], false);
+  _tc = cygwin_shared->tty[get_unit ()];
+
+  tcinit (false);
 
   cygwin_shared->tty.attach (get_unit ());
 
@@ -716,7 +718,7 @@ fhandler_tty_slave::init (HANDLE h, DWORD a, mode_t)
         tty process group leader.
         TODO: Investigate how SIGTTIN should be handled with pure-windows
         programs. */
-      pinfo p (tc->getpgid ());
+      pinfo p (tc ()->getpgid ());
       /* We should only grab this when the process group owner for this
         tty is a non-cygwin process or we've been started directly
         from a non-Cygwin process with no Cygwin ancestry.  */
@@ -724,7 +726,7 @@ fhandler_tty_slave::init (HANDLE h, DWORD a, mode_t)
        {
          termios_printf ("Setting process group leader to %d since %W(%d) is not a cygwin process",
                          myself->pgid, p->progname, p->pid);
-         tc->setpgid (myself->pgid);
+         tc ()->setpgid (myself->pgid);
        }
     }
 
@@ -1248,7 +1250,7 @@ fhandler_tty_slave::fch_open_handles ()
 {
   char buf[MAX_PATH];
 
-  tc = cygwin_shared->tty[get_unit ()];
+  _tc = cygwin_shared->tty[get_unit ()];
   shared_name (buf, INPUT_AVAILABLE_EVENT, get_unit ());
   input_available_event = OpenEvent (READ_CONTROL | WRITE_DAC | WRITE_OWNER,
                                     TRUE, buf);
@@ -1496,7 +1498,7 @@ fhandler_pty_master::write (const void *ptr, size_t len)
 {
   int i;
   char *p = (char *) ptr;
-  termios ti = tc->ti;
+  termios ti = tc ()->ti;
 
   bg_check_types bg = bg_check (SIGTTOU);
   if (bg <= bg_eof)
@@ -1796,8 +1798,9 @@ fhandler_pty_master::setup (bool ispty)
   SECURITY_ATTRIBUTES sa = { sizeof (SECURITY_ATTRIBUTES), NULL, TRUE };
 
   tty& t = *cygwin_shared->tty[get_unit ()];
+  _tc = (tty_min *)&t;
 
-  tcinit (&t, true);           /* Set termios information.  Force initialization. */
+  tcinit (true);               /* Set termios information.  Force initialization. */
 
   const char *errstr = NULL;
   DWORD pipe_mode = PIPE_NOWAIT;

diff --git a/winsup/cygwin/pinfo.cc b/winsup/cygwin/pinfo.cc
index d0ee4be..c817863 100644 (file)
--- a/winsup/cygwin/pinfo.cc
+++ b/winsup/cygwin/pinfo.cc
@@ -403,6 +403,12 @@ _pinfo::set_ctty (tty_min *tc, int flags, fhandler_termios *fh)
       lock_ttys here;
       syscall_printf ("attaching %s sid %d, pid %d, pgid %d, tty->pgid %d, tty->sid %d",
                      __ctty (), sid, pid, pgid, tc->getpgid (), tc->getsid ());
+      if (!cygwin_finished_initializing && !myself->cygstarted
+         && myself->pgid == myself->pid)
+       {
+         myself->pgid = tc->getpgid ();
+         myself->sid = tc->getsid ();
+       }
 
       pinfo p (tc->getsid ());
       if (sid == pid && (!p || p->pid == pid || !p->exists ()))

diff --git a/winsup/cygwin/shared_info.h b/winsup/cygwin/shared_info.h
index 9dd49cb..d1260e6 100644 (file)
--- a/winsup/cygwin/shared_info.h
+++ b/winsup/cygwin/shared_info.h
@@ -79,14 +79,6 @@ void __stdcall shared_destroy ();
             (((DWORD) ((p) + 1) + system_info.dwAllocationGranularity - 1) / \
              system_info.dwAllocationGranularity)))
 
-#ifdef _FHANDLER_H_
-struct console_state
-{
-  tty_min tty_min_state;
-  dev_console dev_state;
-};
-#endif
-
 HANDLE get_shared_parent_dir ();
 HANDLE get_session_parent_dir ();
 char *__stdcall shared_name (char *, const char *, int);

diff --git a/winsup/cygwin/tty.h b/winsup/cygwin/tty.h
index 09583bd..1aa908c 100644 (file)
--- a/winsup/cygwin/tty.h
+++ b/winsup/cygwin/tty.h
@@ -8,6 +8,8 @@ This software is a copyrighted work licensed under the terms of the
 Cygwin license.  Please consult the file "CYGWIN_LICENSE" for
 details. */
 
+#ifndef _TTY_H
+#define _TTY_H
 /* tty tables */
 
 #define INP_BUFFER_SIZE 256
@@ -148,3 +150,4 @@ public:
 };
 
 extern "C" int ttyslot (void);
+#endif /*_TTY_H*/