From: Owen W. Taylor <otaylor@fishsoup.net>
Date: Tue, 18 Aug 2015 19:19:05 +0000 (-0400)
Subject: logind: get a fresh file descriptor to clean up a VT
X-Git-Tag: v226.4~1^2~126
X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?p=elogind.git;a=commitdiff_plain;h=b79f7282da05915ac86a02e460cccad41f7b7d0c

logind: get a fresh file descriptor to clean up a VT

When the controlling process exits, any existing file descriptors
for that FD will be marked as hung-up and ioctls on them will
file with EIO. To work around this, open a new file descriptor
for the VT we want to clean up.

Thanks to Ray Strode for help in sorting out the problem and
coming up with a fix!

https://github.com/systemd/systemd/issues/989
---

diff --git a/src/login/logind-session.c b/src/login/logind-session.c
index 70a93c689..1a5c76b81 100644
--- a/src/login/logind-session.c
+++ b/src/login/logind-session.c
@@ -1045,7 +1045,18 @@ void session_restore_vt(Session *s) {
         int vt, kb = K_XLATE;
         struct vt_mode mode = { 0 };
 
+        /* We need to get a fresh handle to the virtual terminal,
+         * since the old file-descriptor is potentially in a hung-up
+         * state after the controlling process exited; we do a
+         * little dance to avoid having the terminal be available
+         * for reuse before we've cleaned it up.
+         */
+        int old_fd = s->vtfd;
+        s->vtfd = -1;
+
         vt = session_open_vt(s);
+        safe_close(old_fd);
+
         if (vt < 0)
                 return;