diff options
author | kris <kris@FreeBSD.org> | 2001-07-23 22:00:51 +0000 |
---|---|---|
committer | kris <kris@FreeBSD.org> | 2001-07-23 22:00:51 +0000 |
commit | da715e7cac1451ae3fb5369dbdc5b89df5e1ee5c (patch) | |
tree | e5b2bca7ffc1d7e38ba4a0c0205b0c13e35378d0 /libexec/telnetd/utility.c | |
parent | d05113329370acaef185fddba13412afb6e9a48e (diff) | |
download | FreeBSD-src-da715e7cac1451ae3fb5369dbdc5b89df5e1ee5c.zip FreeBSD-src-da715e7cac1451ae3fb5369dbdc5b89df5e1ee5c.tar.gz |
MFcrypto/telnet/telnetd: Correct semantics of output_data*() and netflush()
to ensure deterministic operation
Diffstat (limited to 'libexec/telnetd/utility.c')
-rw-r--r-- | libexec/telnetd/utility.c | 33 |
1 files changed, 18 insertions, 15 deletions
diff --git a/libexec/telnetd/utility.c b/libexec/telnetd/utility.c index 69a61b3..677dadb 100644 --- a/libexec/telnetd/utility.c +++ b/libexec/telnetd/utility.c @@ -62,10 +62,9 @@ static const char rcsid[] = void ttloop() { - void netflush(); DIAG(TD_REPORT, output_data("td: ttloop\r\n")); - if (nfrontp-nbackp) { + if (nfrontp - nbackp > 0) { netflush(); } ncc = read(net, netibuf, sizeof netibuf); @@ -242,10 +241,13 @@ netflush() int n; extern int not42; - if ((n = nfrontp - nbackp) > 0) { + while ((n = nfrontp - nbackp) > 0) { +#if 0 + /* XXX This causes output_data() to recurse and die */ DIAG(TD_REPORT, { n += output_data("td: netflush %d chars\r\n", n); }); +#endif /* * if no urgent data, or if the other side appears to be an * old 4.2 client (and thus unable to survive TCP urgent data), @@ -269,18 +271,19 @@ netflush() n = send(net, nbackp, n, MSG_OOB); /* URGENT data */ } } - } - if (n < 0) { - if (errno == EWOULDBLOCK || errno == EINTR) - return; - cleanup(0); - } - nbackp += n; - if (nbackp >= neturg) { - neturg = 0; - } - if (nbackp == nfrontp) { - nbackp = nfrontp = netobuf; + if (n == -1) { + if (errno == EWOULDBLOCK || errno == EINTR) + continue; + cleanup(0); + /* NOTREACHED */ + } + nbackp += n; + if (nbackp >= neturg) { + neturg = 0; + } + if (nbackp == nfrontp) { + nbackp = nfrontp = netobuf; + } } return; } /* end of netflush */ |