aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDaniel P. Berrange <berrange@redhat.com>2016-03-14 14:15:35 -0400
committerDaniel P. Berrange <berrange@redhat.com>2016-03-15 13:55:52 +0000
commitcfd47a71df51047833d182e9e97244e7816b57da (patch)
tree6978d20e647ce3ca49e62e6cd7fea6e738f2befc
parent618a5a8bc52ba0f2ecbb3dffd01e657f4d841f75 (diff)
downloadqemu-cfd47a71df51047833d182e9e97244e7816b57da.zip
qemu-cfd47a71df51047833d182e9e97244e7816b57da.tar.gz
qemu-cfd47a71df51047833d182e9e97244e7816b57da.tar.bz2
io: stronger check for support for IPv4/6
Instead of just checking for bind(), also check whether getaddrinfo can resolve IPv6 addresses. This catches failure when travis runs QEMU builds inside minimal docker containers Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
-rw-r--r--tests/test-io-channel-socket.c70
1 files changed, 49 insertions, 21 deletions
diff --git a/tests/test-io-channel-socket.c b/tests/test-io-channel-socket.c
index ae665f5..87018ac 100644
--- a/tests/test-io-channel-socket.c
+++ b/tests/test-io-channel-socket.c
@@ -23,44 +23,72 @@
#include "io/channel-util.h"
#include "io-channel-helpers.h"
-static int check_bind(struct sockaddr *sa, socklen_t salen, bool *has_proto)
+#ifndef AI_ADDRCONFIG
+# define AI_ADDRCONFIG 0
+#endif
+#ifndef AI_V4MAPPED
+# define AI_V4MAPPED 0
+#endif
+#ifndef EAI_ADDRFAMILY
+# define EAI_ADDRFAMILY 0
+#endif
+
+static int check_bind(const char *hostname, bool *has_proto)
{
- int fd;
+ int fd = -1;
+ struct addrinfo ai, *res = NULL;
+ int rc;
+ int ret = -1;
+
+ memset(&ai, 0, sizeof(ai));
+ ai.ai_flags = AI_CANONNAME | AI_V4MAPPED | AI_ADDRCONFIG;
+ ai.ai_family = AF_UNSPEC;
+ ai.ai_socktype = SOCK_STREAM;
+
+ /* lookup */
+ rc = getaddrinfo(hostname, NULL, &ai, &res);
+ if (rc != 0) {
+ if (rc == EAI_ADDRFAMILY ||
+ rc == EAI_FAMILY) {
+ *has_proto = false;
+ goto done;
+ }
+ goto cleanup;
+ }
- fd = socket(sa->sa_family, SOCK_STREAM, 0);
+ fd = qemu_socket(res->ai_family, res->ai_socktype, res->ai_protocol);
if (fd < 0) {
- return -1;
+ goto cleanup;
}
- if (bind(fd, sa, salen) < 0) {
- close(fd);
+ if (bind(fd, res->ai_addr, res->ai_addrlen) < 0) {
if (errno == EADDRNOTAVAIL) {
*has_proto = false;
- return 0;
+ goto done;
}
- return -1;
+ goto cleanup;
}
- close(fd);
*has_proto = true;
- return 0;
+ done:
+ ret = 0;
+
+ cleanup:
+ if (fd != -1) {
+ close(fd);
+ }
+ if (res) {
+ freeaddrinfo(res);
+ }
+ return ret;
}
static int check_protocol_support(bool *has_ipv4, bool *has_ipv6)
{
- struct sockaddr_in sin = {
- .sin_family = AF_INET,
- .sin_addr = { .s_addr = htonl(INADDR_LOOPBACK) },
- };
- struct sockaddr_in6 sin6 = {
- .sin6_family = AF_INET6,
- .sin6_addr = IN6ADDR_LOOPBACK_INIT,
- };
-
- if (check_bind((struct sockaddr *)&sin, sizeof(sin), has_ipv4) < 0) {
+ if (check_bind("127.0.0.1", has_ipv4) < 0) {
return -1;
}
- if (check_bind((struct sockaddr *)&sin6, sizeof(sin6), has_ipv6) < 0) {
+ if (check_bind("::1", has_ipv6) < 0) {
return -1;
}