bool master_readable:1;
bool master_writable:1;
bool master_hangup:1;
- bool master_suppressed_hangup:1;
- bool repeat:1;
+ /* Continue reading after hangup? */
+ bool ignore_vhangup:1;
bool last_char_set:1;
char last_char;
k = read(f->master, f->out_buffer + f->out_buffer_full, LINE_MAX - f->out_buffer_full);
if (k < 0) {
- if (errno == EAGAIN)
- f->master_readable = false;
-
- else if (errno == EIO && f->repeat) {
-
- /* Note that EIO on the master device
- * might be cause by vhangup() or
- * temporary closing of everything on
- * the other side, we treat it like
- * EAGAIN here and try again, unless
- * repeat is off. */
+ /* Note that EIO on the master device
+ * might be caused by vhangup() or
+ * temporary closing of everything on
+ * the other side, we treat it like
+ * EAGAIN here and try again, unless
+ * ignore_vhangup is off. */
+ if (errno == EAGAIN || (errno == EIO && f->ignore_vhangup))
f->master_readable = false;
- f->master_suppressed_hangup = true;
-
- } else if (errno == EPIPE || errno == ECONNRESET || errno == EIO) {
+ else if (errno == EPIPE || errno == ECONNRESET || errno == EIO) {
f->master_readable = f->master_writable = false;
f->master_hangup = true;
assert(fd >= 0);
assert(fd == f->master);
- f->master_suppressed_hangup = false;
-
if (revents & (EPOLLIN|EPOLLHUP))
f->master_readable = true;
/* The window size changed, let's forward that. */
if (ioctl(STDOUT_FILENO, TIOCGWINSZ, &ws) >= 0)
- (void)ioctl(f->master, TIOCSWINSZ, &ws);
+ (void) ioctl(f->master, TIOCSWINSZ, &ws);
return 0;
}
-int pty_forward_new(sd_event *event, int master, bool repeat, PTYForward **ret) {
+int pty_forward_new(sd_event *event, int master, bool ignore_vhangup, PTYForward **ret) {
_cleanup_(pty_forward_freep) PTYForward *f = NULL;
struct winsize ws;
int r;
if (!f)
return -ENOMEM;
- f->repeat = repeat;
+ f->ignore_vhangup = ignore_vhangup;
if (event)
f->event = sd_event_ref(event);
return r;
r = sd_event_add_signal(f->event, &f->sigwinch_event_source, SIGWINCH, on_sigwinch_event, f);
+ if (r < 0)
+ return r;
*ret = f;
f = NULL;
return 0;
}
-int pty_forward_set_repeat(PTYForward *f, bool repeat) {
+int pty_forward_set_ignore_vhangup(PTYForward *f, bool ignore_vhangup) {
int r;
assert(f);
- if (f->repeat == repeat)
+ if (f->ignore_vhangup == ignore_vhangup)
return 0;
- f->repeat = repeat;
-
- /* Are we currently in a suppress hangup phase? If so, let's
- * immediately terminate things */
- if (!f->repeat && f->master_suppressed_hangup) {
+ f->ignore_vhangup = ignore_vhangup;
+ if (!f->ignore_vhangup) {
- /* Let's try to read again from the master fd, and if
- * it is this will now cause termination of the
- * session. */
+ /* We shall now react to vhangup()s? Let's check
+ * immediately if we might be in one */
f->master_readable = true;
r = shovel(f);
return 0;
}
-int pty_forward_get_repeat(PTYForward *f) {
+int pty_forward_get_ignore_vhangup(PTYForward *f) {
assert(f);
- return f->repeat;
+ return f->ignore_vhangup;
}