diff options
author | Denis Vlasenko <vda.linux@googlemail.com> | 2008-07-21 13:22:28 +0400 |
---|---|---|
committer | Denis Vlasenko <vda.linux@googlemail.com> | 2008-07-21 13:22:28 +0400 |
commit | 1101d1c98068d7a49942d6e9e11ca2e789a9cdba (patch) | |
tree | 8fcf79323f16e9e29ad0e257184ce4714b29c23b /networking/telnet.c | |
parent | 84c8daa11f4c656bb4ba50b6e1c6d3b34c0f1b39 (diff) |
telnet: shrink
telnetd: shrink, and fix issue file printing
test: better and shorter usage text
function old new delta
putiac2 51 50 -1
putiac 24 20 -4
handlenetoutput 95 91 -4
telnet_main 1480 1475 -5
iacflush 37 32 -5
make_new_session 436 421 -15
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 0/6 up/down: 0/-34) Total: -34 bytes
Diffstat (limited to 'networking/telnet.c')
-rw-r--r-- | networking/telnet.c | 32 |
1 files changed, 15 insertions, 17 deletions
diff --git a/networking/telnet.c b/networking/telnet.c index aaaa2648f..63ee38088 100644 --- a/networking/telnet.c +++ b/networking/telnet.c @@ -52,9 +52,10 @@ enum { typedef unsigned char byte; +enum { netfd = 3 }; + struct globals { - int netfd; /* console fd:s are 0 and 1 (and 2) */ - short iaclen; /* could even use byte */ + int iaclen; /* could even use byte, but it's a loss on x86 */ byte telstate; /* telnet negotiation state from network input */ byte telwish; /* DO, DONT, WILL, WONT */ byte charmode; @@ -95,7 +96,7 @@ static int subneg(byte c); static void iacflush(void) { - write(G.netfd, G.iacbuf, G.iaclen); + write(netfd, G.iacbuf, G.iaclen); G.iaclen = 0; } @@ -191,7 +192,7 @@ static void handlenetoutput(int len) outbuf[j++] = 0x00; } if (j > 0) - write(G.netfd, outbuf, j); + write(netfd, outbuf, j); } static void handlenetinput(int len) @@ -545,7 +546,7 @@ static void cookmode(void) #define USE_POLL 1 int telnet_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE; -int telnet_main(int argc, char **argv) +int telnet_main(int argc UNUSED_PARAM, char **argv) { char *host; int port; @@ -573,9 +574,6 @@ int telnet_main(int argc, char **argv) cfmakeraw(&G.termios_raw); } - if (argc < 2) - bb_show_usage(); - #if ENABLE_FEATURE_TELNET_AUTOLOGIN if (1 & getopt32(argv, "al:", &G.autologin)) G.autologin = getenv("USER"); @@ -590,20 +588,20 @@ int telnet_main(int argc, char **argv) if (*argv) /* extra params?? */ bb_show_usage(); - G.netfd = create_and_connect_stream_or_die(host, port); + xmove_fd(create_and_connect_stream_or_die(host, port), netfd); - setsockopt(G.netfd, SOL_SOCKET, SO_KEEPALIVE, &const_int_1, sizeof(const_int_1)); + setsockopt(netfd, SOL_SOCKET, SO_KEEPALIVE, &const_int_1, sizeof(const_int_1)); signal(SIGINT, fgotsig); #ifdef USE_POLL - ufds[0].fd = 0; ufds[1].fd = G.netfd; + ufds[0].fd = 0; ufds[1].fd = netfd; ufds[0].events = ufds[1].events = POLLIN; #else FD_ZERO(&readfds); FD_SET(STDIN_FILENO, &readfds); - FD_SET(G.netfd, &readfds); - maxfd = G.netfd + 1; + FD_SET(netfd, &readfds); + maxfd = netfd + 1; #endif while (1) { @@ -642,17 +640,17 @@ int telnet_main(int argc, char **argv) #ifdef USE_POLL if (ufds[1].revents) /* well, should check POLLIN, but ... */ #else - if (FD_ISSET(G.netfd, &rfds)) + if (FD_ISSET(netfd, &rfds)) #endif { - len = read(G.netfd, G.buf, DATABUFSIZE); + len = read(netfd, G.buf, DATABUFSIZE); if (len <= 0) { write_str(1, "Connection closed by foreign host\r\n"); doexit(EXIT_FAILURE); } - TRACE(0, ("Read netfd (%d): %d\n", G.netfd, len)); + TRACE(0, ("Read netfd (%d): %d\n", netfd, len)); handlenetinput(len); } } - } + } /* while (1) */ } |