diff options
Diffstat (limited to 'alacritty_terminal')
| -rw-r--r-- | alacritty_terminal/src/tty/windows/conpty.rs | 4 | ||||
| -rw-r--r-- | alacritty_terminal/src/tty/windows/mod.rs | 2 |
2 files changed, 6 insertions, 0 deletions
diff --git a/alacritty_terminal/src/tty/windows/conpty.rs b/alacritty_terminal/src/tty/windows/conpty.rs index 31f4c252..a1c7edfd 100644 --- a/alacritty_terminal/src/tty/windows/conpty.rs +++ b/alacritty_terminal/src/tty/windows/conpty.rs @@ -90,6 +90,10 @@ pub type ConptyHandle = Arc<Conpty>; impl Drop for Conpty { fn drop(&mut self) { + // XXX: This will block until the conout pipe is drained. Will cause a deadlock if the + // conout pipe has already been dropped by this point. + // + // See PR #3084 and https://docs.microsoft.com/en-us/windows/console/closepseudoconsole unsafe { (self.api.ClosePseudoConsole)(self.handle) } } } diff --git a/alacritty_terminal/src/tty/windows/mod.rs b/alacritty_terminal/src/tty/windows/mod.rs index c9c0be73..e112d305 100644 --- a/alacritty_terminal/src/tty/windows/mod.rs +++ b/alacritty_terminal/src/tty/windows/mod.rs @@ -45,6 +45,8 @@ pub enum PtyHandle { } pub struct Pty { + // XXX: Handle is required to be the first field, to ensure correct drop order. Dropping + // `conout` before `handle` will cause a deadlock. handle: PtyHandle, // TODO: It's on the roadmap for the Conpty API to support Overlapped I/O. // See https://github.com/Microsoft/console/issues/262 |