X-Git-Url: https://mattmccutchen.net/rsync/rsync.git/blobdiff_plain/5d2640376e3b5b21f6049161b55baf6f4e2acce8..0cd2f40764dd156de309f7536f3867968b6c0fdd:/socket.c diff --git a/socket.c b/socket.c index 88228046..9ac182d9 100644 --- a/socket.c +++ b/socket.c @@ -33,17 +33,6 @@ #include "rsync.h" -static const char default_name[] = "UNKNOWN"; - -static int lookup_name(const struct sockaddr_storage *ss, - socklen_t ss_len, - char *name_buf, size_t name_buf_len, - char *port_buf, size_t port_buf_len); - -static int check_name(const struct sockaddr_storage *ss, - socklen_t ss_len, - char *name_buf, - const char *port_buf); /* Establish a proxy connection on an open socket to a web roxy by * using the CONNECT method. */ @@ -53,7 +42,7 @@ static int establish_proxy_connection(int fd, char *host, int port) char *cp; snprintf(buffer, sizeof(buffer), "CONNECT %s:%d HTTP/1.0\r\n\r\n", host, port); - if (write(fd, buffer, strlen(buffer)) != strlen(buffer)) { + if (write(fd, buffer, strlen(buffer)) != (int) strlen(buffer)) { rprintf(FERROR, "failed to write to proxy: %s\n", strerror(errno)); return -1; @@ -577,178 +566,6 @@ void become_daemon(void) } } -/** - * Return the IP addr of the client as a string - **/ -char *client_addr(int fd) -{ - struct sockaddr_storage ss; - socklen_t length = sizeof ss; - static char addr_buf[100]; - static int initialised; - - if (initialised) return addr_buf; - - initialised = 1; - - if (getpeername(fd, (struct sockaddr *)&ss, &length)) { - exit_cleanup(RERR_SOCKETIO); - } - - getnameinfo((struct sockaddr *)&ss, length, - addr_buf, sizeof(addr_buf), NULL, 0, NI_NUMERICHOST); - return addr_buf; -} - - -static int get_sockaddr_family(const struct sockaddr_storage *ss) -{ - return ((struct sockaddr *) ss)->sa_family; -} - - -/** - * Return the DNS name of the client. - * - * The name is statically cached so that repeated lookups are quick, - * so there is a limit of one lookup per customer. - * - * If anything goes wrong, including the name->addr->name check, then - * we just use "UNKNOWN", so you can use that value in hosts allow - * lines. - **/ -char *client_name(int fd) -{ - struct sockaddr_storage ss; - socklen_t ss_len = sizeof ss; - static char name_buf[100]; - static char port_buf[100]; - static int initialised; - - if (initialised) return name_buf; - - strcpy(name_buf, default_name); - initialised = 1; - - if (getpeername(fd, (struct sockaddr *)&ss, &ss_len)) { - /* FIXME: Can we really not continue? */ - rprintf(FERROR, RSYNC_NAME ": getpeername on fd%d failed: %s\n", - fd, strerror(errno)); - exit_cleanup(RERR_SOCKETIO); - } - - if (!lookup_name(&ss, ss_len, name_buf, sizeof name_buf, port_buf, sizeof port_buf)) - check_name(&ss, ss_len, name_buf, port_buf); - - return name_buf; -} - - -/** - * Look up a name from @p ss into @p name_buf. - **/ -static int lookup_name(const struct sockaddr_storage *ss, - socklen_t ss_len, - char *name_buf, size_t name_buf_len, - char *port_buf, size_t port_buf_len) -{ - int name_err; - -#ifdef INET6 - if (get_sockaddr_family(ss) == AF_INET6 && - IN6_IS_ADDR_V4MAPPED(&((struct sockaddr_in6 *)ss)->sin6_addr)) { - /* OK, so ss is in the IPv6 family, but it is really - * an IPv4 address: something like - * "::ffff:10.130.1.2". If we use it as-is, then the - * reverse lookup might fail or perhaps something else - * bad might happen. So instead we convert it to an - * equivalent address in the IPv4 address family. */ - struct sockaddr_in6 sin6; - struct sockaddr_in *sin; - - memcpy(&sin6, ss, sizeof(sin6)); - sin = (struct sockaddr_in *)ss; - memset(sin, 0, sizeof(*sin)); - sin->sin_family = AF_INET; - ss_len = sizeof(struct sockaddr_in); -#ifdef HAVE_SOCKADDR_LEN - sin->sin_len = ss_len; -#endif - sin->sin_port = sin6.sin6_port; - /* FIXME: Isn't there a macro we can use here rather - * than grovelling through the struct? It might be - * wrong on some systems. */ - memcpy(&sin->sin_addr, &sin6.sin6_addr.s6_addr[12], - sizeof(sin->sin_addr)); - } -#endif - - /* reverse lookup */ - name_err = getnameinfo((struct sockaddr *) ss, ss_len, - name_buf, name_buf_len, - port_buf, port_buf_len, - NI_NAMEREQD | NI_NUMERICSERV); - if (name_err != 0) { - strcpy(name_buf, default_name); - rprintf(FERROR, RSYNC_NAME ": name lookup failed: %s\n", - gai_strerror(name_err)); - return name_err; - } - - return 0; -} - - - -/* Do a forward lookup on name_buf and make sure it corresponds to ss - * -- otherwise we may be being spoofed. If we suspect we are, then - * we don't abort the connection but just emit a warning. */ -static int check_name(const struct sockaddr_storage *ss, - socklen_t ss_len, - char *name_buf, - const char *port_buf) -{ - struct addrinfo hints, *res, *res0; - int error; - - memset(&hints, 0, sizeof(hints)); - hints.ai_family = PF_UNSPEC; - hints.ai_flags = AI_CANONNAME; - hints.ai_socktype = SOCK_STREAM; - error = getaddrinfo(name_buf, port_buf, &hints, &res0); - if (error) { - strcpy(name_buf, default_name); - rprintf(FERROR, - RSYNC_NAME ": forward name lookup for %s:%s failed: %s\n", - name_buf, port_buf, - gai_strerror(error)); - return error; - } - - - /* We expect that one of the results will be the same as ss. */ - for (res = res0; res; res = res->ai_next) { - if (res->ai_family != get_sockaddr_family(ss)) - continue; - if (res->ai_addrlen != ss_len) - continue; - if (memcmp(res->ai_addr, ss, res->ai_addrlen) == 0) - break; - } - - if (res == NULL) { - strcpy(name_buf, default_name); - /* We hit the end of the list without finding an - * address that was the same as ss. */ - rprintf(FERROR, RSYNC_NAME - ": no address record for \"%s\" corresponds to peer name: spoofed address?\n", - name_buf); - } - - freeaddrinfo(res0); - return 0; -} - /******************************************************************* this is like socketpair but uses tcp. It is used by the Samba @@ -820,8 +637,8 @@ static int socketpair_tcp(int fd[2]) /** * Run a program on a local tcp socket, so that we can talk to it's - * stdin and stdout. This is used to launch ssh and similar - * connection helper programs for rsync. + * stdin and stdout. This is used to fake a connection to a daemon + * for testing -- not for the normal case of running SSH. * * @return a socket which is attached to a subprocess running * "prog". stdin and stdout are attached. stderr is left attached to @@ -830,12 +647,6 @@ static int socketpair_tcp(int fd[2]) int sock_exec(const char *prog) { int fd[2]; - - if (verbose > 0) { - rprintf(FINFO, RSYNC_NAME - ": open connection using \"%s\"\n", - prog); - } if (socketpair_tcp(fd) != 0) { rprintf (FERROR, RSYNC_NAME