=================================================================== RCS file: /cvsrepo/anoncvs/cvs/src/usr.bin/ssh/canohost.c,v retrieving revision 1.38.2.2 retrieving revision 1.39 diff -u -r1.38.2.2 -r1.39 --- src/usr.bin/ssh/canohost.c 2005/03/10 17:15:04 1.38.2.2 +++ src/usr.bin/ssh/canohost.c 2004/03/31 21:58:47 1.39 @@ -12,7 +12,7 @@ */ #include "includes.h" -RCSID("$OpenBSD: canohost.c,v 1.38.2.2 2005/03/10 17:15:04 brad Exp $"); +RCSID("$OpenBSD: canohost.c,v 1.39 2004/03/31 21:58:47 djm Exp $"); #include "packet.h" #include "xmalloc.h" @@ -27,7 +27,7 @@ */ static char * -get_remote_hostname(int sock, int use_dns) +get_remote_hostname(int socket, int use_dns) { struct sockaddr_storage from; int i; @@ -38,13 +38,13 @@ /* Get IP address of client. */ fromlen = sizeof(from); memset(&from, 0, sizeof(from)); - if (getpeername(sock, (struct sockaddr *)&from, &fromlen) < 0) { + if (getpeername(socket, (struct sockaddr *)&from, &fromlen) < 0) { debug("getpeername failed: %.100s", strerror(errno)); cleanup_exit(255); } if (from.ss_family == AF_INET) - check_ip_options(sock, ntop); + check_ip_options(socket, ntop); if (getnameinfo((struct sockaddr *)&from, fromlen, ntop, sizeof(ntop), NULL, 0, NI_NUMERICHOST) != 0) @@ -132,7 +132,7 @@ */ /* IPv4 only */ static void -check_ip_options(int sock, char *ipaddr) +check_ip_options(int socket, char *ipaddr) { u_char options[200]; char text[sizeof(options) * 3 + 1]; @@ -145,7 +145,7 @@ else ipproto = IPPROTO_IP; option_size = sizeof(options); - if (getsockopt(sock, ipproto, IP_OPTIONS, options, + if (getsockopt(socket, ipproto, IP_OPTIONS, options, &option_size) >= 0 && option_size != 0) { text[0] = '\0'; for (i = 0; i < option_size; i++) @@ -194,60 +194,58 @@ * The returned string must be freed. */ static char * -get_socket_address(int sock, int remote, int flags) +get_socket_address(int socket, int remote, int flags) { struct sockaddr_storage addr; socklen_t addrlen; char ntop[NI_MAXHOST]; - int r; /* Get IP address of client. */ addrlen = sizeof(addr); memset(&addr, 0, sizeof(addr)); if (remote) { - if (getpeername(sock, (struct sockaddr *)&addr, &addrlen) + if (getpeername(socket, (struct sockaddr *)&addr, &addrlen) < 0) return NULL; } else { - if (getsockname(sock, (struct sockaddr *)&addr, &addrlen) + if (getsockname(socket, (struct sockaddr *)&addr, &addrlen) < 0) return NULL; } /* Get the address in ascii. */ - if ((r = getnameinfo((struct sockaddr *)&addr, addrlen, ntop, - sizeof(ntop), NULL, 0, flags)) != 0) { - error("get_socket_address: getnameinfo %d failed: %s", flags, - r == EAI_SYSTEM ? strerror(errno) : gai_strerror(r)); + if (getnameinfo((struct sockaddr *)&addr, addrlen, ntop, sizeof(ntop), + NULL, 0, flags) != 0) { + error("get_socket_address: getnameinfo %d failed", flags); return NULL; } return xstrdup(ntop); } char * -get_peer_ipaddr(int sock) +get_peer_ipaddr(int socket) { char *p; - if ((p = get_socket_address(sock, 1, NI_NUMERICHOST)) != NULL) + if ((p = get_socket_address(socket, 1, NI_NUMERICHOST)) != NULL) return p; return xstrdup("UNKNOWN"); } char * -get_local_ipaddr(int sock) +get_local_ipaddr(int socket) { char *p; - if ((p = get_socket_address(sock, 0, NI_NUMERICHOST)) != NULL) + if ((p = get_socket_address(socket, 0, NI_NUMERICHOST)) != NULL) return p; return xstrdup("UNKNOWN"); } char * -get_local_name(int sock) +get_local_name(int socket) { - return get_socket_address(sock, 0, NI_NAMEREQD); + return get_socket_address(socket, 0, NI_NAMEREQD); } /* @@ -294,7 +292,6 @@ struct sockaddr_storage from; socklen_t fromlen; char strport[NI_MAXSERV]; - int r; /* Get IP address of client. */ fromlen = sizeof(from); @@ -311,10 +308,9 @@ } } /* Return port number. */ - if ((r = getnameinfo((struct sockaddr *)&from, fromlen, NULL, 0, - strport, sizeof(strport), NI_NUMERICSERV)) != 0) - fatal("get_sock_port: getnameinfo NI_NUMERICSERV failed: %s", - r == EAI_SYSTEM ? strerror(errno) : gai_strerror(r)); + if (getnameinfo((struct sockaddr *)&from, fromlen, NULL, 0, + strport, sizeof(strport), NI_NUMERICSERV) != 0) + fatal("get_sock_port: getnameinfo NI_NUMERICSERV failed"); return atoi(strport); } @@ -343,13 +339,7 @@ int get_remote_port(void) { - static int port = -1; - - /* Cache to avoid getpeername() on a dead connection */ - if (port == -1) - port = get_port(0); - - return port; + return get_port(0); } int