X-Git-Url: https://git.distorted.org.uk/u/mdw/putty/blobdiff_plain/8ce72d2c1c7651def2425be607ef623e6b0b9bd2..c9def1b8e8960113ab4fd91d0b69d5b422cf339f:/telnet.c diff --git a/telnet.c b/telnet.c index 399c1260..1b83a311 100644 --- a/telnet.c +++ b/telnet.c @@ -133,10 +133,7 @@ static struct Opt *opts[] = { &o_we_sga, &o_they_sga, NULL }; -#if 0 static int in_synch; -#endif - static int sb_opt, sb_len; static char *sb_buf = NULL; static int sb_size = 0; @@ -168,16 +165,6 @@ static void s_write (void *buf, int len) { try_write(); } -static void c_write (char *buf, int len) { - while (len--) { - int new_head = (inbuf_head + 1) & INBUF_MASK; - if (new_head != inbuf_reap) { - inbuf[inbuf_head] = *buf++; - inbuf_head = new_head; - } - } -} - static void log_option (char *sender, int cmd, int option) { char buf[50]; sprintf(buf, "%s:\t%s %s", sender, @@ -213,6 +200,8 @@ static void activate_option (struct Opt *o) { */ deactivate_option (o->option==TELOPT_NEW_ENVIRON ? &o_oenv : &o_nenv); } + if (o->option == TELOPT_ECHO && cfg.ldisc_term) + ldisc = &ldisc_simple; } static void refused_option (struct Opt *o) { @@ -221,6 +210,8 @@ static void refused_option (struct Opt *o) { send_opt (WILL, TELOPT_OLD_ENVIRON); o_oenv.state = REQUESTED; } + if (o->option == TELOPT_ECHO && cfg.ldisc_term) + ldisc = &ldisc_term; } static void proc_rec_opt (int cmd, int option) { @@ -380,7 +371,6 @@ static enum { } telnet_state = TOPLEVEL; static void do_telnet_read (char *buf, int len) { - unsigned char b[10]; while (len--) { int c = (unsigned char) *buf++; @@ -393,11 +383,20 @@ static void do_telnet_read (char *buf, int len) { else if (c == IAC) telnet_state = SEENIAC; else { - b[0] = c; -#if 0 if (!in_synch) + c_write1(c); + +#if 1 + /* I can't get the F***ing winsock to insert the urgent IAC + * into the right position! Even with SO_OOBINLINE it gives + * it to recv too soon. And of course the DM byte (that + * arrives in the same packet!) appears several K later!! + * + * Oh well, we do get the DM in the right place so I'll + * just stop hiding on the next 0xf2 and hope for the best. + */ + else if (c == DM) in_synch = 0; #endif - c_write (b, 1); if (c == CR) telnet_state = SEENCR; else @@ -410,10 +409,15 @@ static void do_telnet_read (char *buf, int len) { else if (c == WILL) telnet_state = SEENWILL; else if (c == WONT) telnet_state = SEENWONT; else if (c == SB) telnet_state = SEENSB; + else if (c == DM) { + in_synch = 0; + telnet_state = TOPLEVEL; + } else { - /* ignore (and print) everything else */ - b[0] = c; - c_write(b,1); + /* ignore everything else; print it if it's IAC */ + if (c == IAC) { + c_write1(c); + } telnet_state = TOPLEVEL; } break; @@ -447,8 +451,8 @@ static void do_telnet_read (char *buf, int len) { char *newbuf; sb_size += SB_DELTA; newbuf = (sb_buf ? - realloc(sb_buf, sb_size) : - malloc(sb_size)); + srealloc(sb_buf, sb_size) : + smalloc(sb_size)); if (newbuf) sb_buf = newbuf; else @@ -517,12 +521,10 @@ static char *telnet_init (HWND hwnd, char *host, int port, char **realhost) { default: return "socket(): unknown error"; } -#if 0 { BOOL b = TRUE; setsockopt (s, SOL_SOCKET, SO_OOBINLINE, (void *)&b, sizeof(b)); } -#endif /* * Bind to local address. @@ -560,6 +562,15 @@ static char *telnet_init (HWND hwnd, char *host, int port, char **realhost) { /* * Initialise option states. */ + if( cfg.ldisc_term ) + { + struct Opt **o; + + for (o = opts; *o; o++) + if ((*o)->state == REQUESTED) + (*o)->state = INACTIVE; + } + else { struct Opt **o; @@ -568,13 +579,10 @@ static char *telnet_init (HWND hwnd, char *host, int port, char **realhost) { send_opt ((*o)->send, (*o)->option); } -#if 0 /* * Set up SYNCH state. */ in_synch = FALSE; -#endif - return NULL; } @@ -584,7 +592,11 @@ static char *telnet_init (HWND hwnd, char *host, int port, char **realhost) { */ static int telnet_msg (WPARAM wParam, LPARAM lParam) { int ret; - char buf[256]; + /* This needs to be larger than the packet size now that inbuf + * cannot overflow, in fact the fewer calls we make to windows + * the faster we will run! + */ + char buf[16384]; /* * Because reading less than the whole of the available pending @@ -602,39 +614,27 @@ static int telnet_msg (WPARAM wParam, LPARAM lParam) { switch (WSAGETSELECTEVENT(lParam)) { case FD_READ: case FD_CLOSE: - ret = recv(s, buf, sizeof(buf), 0); - if (ret < 0 && WSAGetLastError() == WSAEWOULDBLOCK) - return 1; - if (ret < 0) /* any _other_ error */ - return -10000-WSAGetLastError(); - if (ret == 0) { - s = INVALID_SOCKET; - return 0; - } -#if 0 - if (in_synch) { - BOOL i; - if (ioctlsocket (s, SIOCATMARK, &i) < 0) { + { + int clear_of_oob = 1; + if (ioctlsocket (s, SIOCATMARK, &clear_of_oob) < 0 ) return -20000-WSAGetLastError(); - } - if (i) - in_synch = FALSE; + + in_synch = !clear_of_oob; + + do { + ret = recv(s, buf, sizeof(buf), 0); + if (ret < 0 && WSAGetLastError() == WSAEWOULDBLOCK) + return 1; + if (ret < 0) /* any _other_ error */ + return -10000-WSAGetLastError(); + if (ret == 0) { + s = INVALID_SOCKET; + return 0; + } + + do_telnet_read (buf, ret); + } while (in_synch); } -#endif - do_telnet_read (buf, ret); - return 1; - case FD_OOB: - do { - ret = recv(s, buf, sizeof(buf), 0); - } while (ret > 0); - telnet_state = TOPLEVEL; - do { - ret = recv(s, buf, 1, MSG_OOB); - if (ret > 0) - do_telnet_read (buf, ret); - } while (ret > 0); - if (ret < 0 && WSAGetLastError() != WSAEWOULDBLOCK) - return -30000-WSAGetLastError(); return 1; case FD_WRITE: if (outbuf_head != outbuf_reap) @@ -651,6 +651,7 @@ static void telnet_send (char *buf, int len) { char *p; static unsigned char iac[2] = { IAC, IAC }; static unsigned char cr[2] = { CR, NUL }; + static unsigned char nl[2] = { CR, LF }; if (s == INVALID_SOCKET) return; @@ -663,7 +664,7 @@ static void telnet_send (char *buf, int len) { s_write (q, p-q); while (p < buf+len && !iswritable((unsigned char)*p)) { - s_write ((unsigned char)*p == IAC ? iac : cr, 2); + s_write ((unsigned char)*p == IAC ? iac : nl, 2); p++; } } @@ -714,8 +715,20 @@ static void telnet_special (Telnet_Special code) { case TS_EOF: b[1] = xEOF; s_write (b, 2); break; case TS_SYNCH: outbuf_head = outbuf_reap = 0; - b[0] = DM; - send (s, b, 1, MSG_OOB); + b[1] = DM; + send (s, b, 2, MSG_OOB); + break; + case TS_RECHO: + if (o_echo.state == INACTIVE || o_echo.state == REALLY_INACTIVE) { + o_echo.state = REQUESTED; + send_opt (o_echo.send, o_echo.option); + } + break; + case TS_LECHO: + if (o_echo.state == ACTIVE) { + o_echo.state = REQUESTED; + send_opt (o_echo.nsend, o_echo.option); + } break; } }