#include "lib/addrinfo.h"
#endif
-// extern int af; /* NO MORE BLOODY GLOBALS! */
-
/* Establish a proxy connection on an open socket to a web roxy by
* using the CONNECT method. */
static int establish_proxy_connection(int fd, char *host, int port)
/**
* Open a socket of the specified type, port and address for incoming data
*
+ * Try to be better about handling the results of getaddrinfo(): when
+ * opening an inbound socket, we might get several address results,
+ * e.g. for the machine's ipv4 and ipv6 name.
+ *
+ * If binding a wildcard, then any one of them should do. If an address
+ * was specified but it's insufficiently specific then that's not our
+ * fault.
+ *
+ * However, some of the advertized addresses may not work because e.g. we
+ * don't have IPv6 support in the kernel. In that case go on and try all
+ * addresses until one succeeds.
+ *
* @param bind_address Local address to bind, or NULL to allow it to
* default.
**/
{
int one=1;
int s;
- struct addrinfo hints, *res;
+ struct addrinfo hints, *res, *resp;
char portbuf[10];
int error;
bind_address, gai_strerror(error));
return -1;
}
- if (res->ai_next) {
- rprintf(FERROR, RSYNC_NAME ": getaddrinfo: bind address %s: "
- "resolved to multiple hosts\n",
- bind_address);
- freeaddrinfo(res);
- return -1;
- }
+ /* XXX: Do we need to care about getting multiple results
+ * back? I think probably not; if the user passed
+ * bind_address == NULL and we set AI_PASSIVE then we ought to
+ * get a wildcard result. */
- s = socket(res->ai_family, res->ai_socktype, res->ai_protocol);
- if (s < 0) {
- rprintf(FERROR, RSYNC_NAME ": open socket in failed: %s\n",
- strerror(errno));
- freeaddrinfo(res);
- return -1;
+ resp = res;
+ while (1) {
+ s = socket(resp->ai_family, resp->ai_socktype, resp->ai_protocol);
+
+ if (s >= 0) {
+ break; /* got a socket */
+ } else if ((resp = resp->ai_next)) {
+ switch (errno) {
+ case EPROTONOSUPPORT:
+ case EAFNOSUPPORT:
+ case EPFNOSUPPORT:
+ /* See if there's another address that will work... */
+ continue;
+ }
+ }
+
+ rprintf(FERROR, RSYNC_NAME ": open inbound socket"
+ "(dom=%d, type=%d, proto=%d) failed: %s\n",
+ resp->ai_family, resp->ai_socktype, resp->ai_protocol,
+ strerror(errno));
+ goto fail;
}
setsockopt(s,SOL_SOCKET,SO_REUSEADDR,(char *)&one,sizeof(one));
/* now we've got a socket - we need to bind it */
if (bind(s, res->ai_addr, res->ai_addrlen) < 0) {
rprintf(FERROR, RSYNC_NAME ": bind failed on port %d\n", port);
- freeaddrinfo(res);
- close(s);
- return -1;
+ close(s);
+ goto fail;
}
return s;
+
+fail:
+ freeaddrinfo(res);
+ return -1;
}
if (getpeername(fd, (struct sockaddr *)&ss, &length)) {
/* FIXME: Can we really not continue? */
rprintf(FERROR, RSYNC_NAME ": getpeername on fd%d failed: %s\n",
- strerror(errno));
+ fd, strerror(errno));
exit_cleanup(RERR_SOCKETIO);
}
#ifdef INET6
- if (ss.ss_family == AF_INET6 &&
+ if (ss.sa_family == AF_INET6 &&
IN6_IS_ADDR_V4MAPPED(&((struct sockaddr_in6 *)&ss)->sin6_addr)) {
struct sockaddr_in6 sin6;
struct sockaddr_in *sin;
break;
}
- /* TODO: Do a forward lookup as well to prevent spoofing */
+ /* TODO: Do a forward lookup as well to prevent spoofing */
if (res == NULL) {
strcpy(name_buf, def);
return name_buf;
}
-/**
- * Convert a string to an IP address. The string can be a name or
- * dotted decimal number.
- *
- * Returns a pointer to a static in_addr struct -- if you call this
- * more than once then you should copy it.
- *
- * TODO: Use getaddrinfo() instead, or make this function call getnameinfo
- **/
-struct in_addr *ip_address(const char *str)
-{
- static struct in_addr ret;
- struct hostent *hp;
-
- if (!str) {
- rprintf (FERROR, "ip_address received NULL name\n");
- return NULL;
- }
-
- /* try as an IP address */
- if (inet_aton(str, &ret) != 0) {
- return &ret;
- }
-
- /* otherwise assume it's a network name of some sort and use
- gethostbyname */
- if ((hp = gethostbyname (str)) == 0) {
- rprintf(FERROR, "gethostbyname failed for \"%s\": unknown host?\n",str);
- return NULL;
- }
-
- if (hp->h_addr == NULL) {
- rprintf(FERROR, "gethostbyname: host address is invalid for host \"%s\"\n",str);
- return NULL;
- }
-
- if (hp->h_length > sizeof ret) {
- rprintf(FERROR, "gethostbyname: host address for \"%s\" is too large\n",
- str);
- return NULL;
- }
-
- if (hp->h_addrtype != AF_INET) {
- rprintf (FERROR, "gethostname: host address for \"%s\" is not IPv4\n",
- str);
- return NULL;
- }
-
- /* This is kind of difficult. The only field in ret is
- s_addr, which is the IP address as a 32-bit int. On
- UNICOS, s_addr is in fact a *bitfield* for reasons best
- know to Cray. This means we can't memcpy in to it. On the
- other hand, h_addr is a char*, so we can't just assign.
-
- Since there's meant to be only one field inside the in_addr
- structure we will try just copying over the top and see how
- that goes. */
- memcpy (&ret, hp->h_addr, hp->h_length);
-
- return &ret;
-}
-
-
/*******************************************************************
this is like socketpair but uses tcp. It is used by the Samba