* [Qemu-devel] [PATCH] sockets: Normalize test for addrinfo flag AI_V4MAPPED
@ 2017-11-02 16:56 Antonio Huete Jiménez
2017-11-02 17:09 ` Daniel P. Berrange
0 siblings, 1 reply; 4+ messages in thread
From: Antonio Huete Jiménez @ 2017-11-02 16:56 UTC (permalink / raw)
To: qemu-devel
From 2b4d9d8cb617445af8f3b062f917dfea42dbdc27 Mon Sep 17 00:00:00 2001
From: Antonio Huete Jimenez <tuxillo@quantumachine.net>
Date: Thu, 2 Nov 2017 17:46:24 +0100
Subject: [PATCH] sockets: Normalize test for addrinfo flag AI_V4MAPPED
Signed-off-by: Antonio Huete Jimenez <tuxillo@quantumachine.net>
---
util/qemu-sockets.c | 54
+++++++++++++++++++++++++++++++++--------------------
1 file changed, 34 insertions(+), 20 deletions(-)
diff --git a/util/qemu-sockets.c b/util/qemu-sockets.c
index b47fb45885..ce35b6a998 100644
--- a/util/qemu-sockets.c
+++ b/util/qemu-sockets.c
@@ -44,7 +44,6 @@
# define AI_NUMERICSERV 0
#endif
-
static int inet_getport(struct addrinfo *e)
{
struct sockaddr_in *i4;
@@ -149,6 +148,31 @@ int inet_ai_family_from_address(InetSocketAddress *addr,
return PF_UNSPEC;
}
+static int
+check_ai_v4mapped(const char *host, const char *port, struct addrinfo *ai,
+ struct addrinfo *res)
+{
+ static int useV4Mapped = -1;
+ int rc;
+
+ /* At least FreeBSD and OS-X 10.6 declare AI_V4MAPPED but
+ * then don't implement it in their getaddrinfo().
+ * Unconditionally deselect AI_V4MAPPED option upon
+ * getaddrinfo() failure, the next call to it will have to
+ * do the error handling.
+ */
+ if (atomic_read(&useV4Mapped) == -1) {
+ rc = getaddrinfo(host, port, ai, &res);
+ if (rc == 0 && (ai->ai_flags & AI_V4MAPPED)) {
+ atomic_set(&useV4Mapped, 1);
+ } else {
+ atomic_set(&useV4Mapped, 0);
+ }
+ }
+
+ return useV4Mapped;
+}
+
static int create_fast_reuse_socket(struct addrinfo *e)
{
int slisten = qemu_socket(e->ai_family, e->ai_socktype, e->ai_protocol);
@@ -378,14 +402,10 @@ static struct addrinfo
*inet_parse_connect_saddr(InetSocketAddress *saddr,
struct addrinfo ai, *res;
int rc;
Error *err = NULL;
- static int useV4Mapped = 1;
memset(&ai, 0, sizeof(ai));
- ai.ai_flags = AI_CANONNAME | AI_ADDRCONFIG;
- if (atomic_read(&useV4Mapped)) {
- ai.ai_flags |= AI_V4MAPPED;
- }
+ ai.ai_flags = AI_CANONNAME | AI_ADDRCONFIG | AI_V4MAPPED;
ai.ai_family = inet_ai_family_from_address(saddr, &err);
ai.ai_socktype = SOCK_STREAM;
@@ -399,21 +419,11 @@ static struct addrinfo
*inet_parse_connect_saddr(InetSocketAddress *saddr,
return NULL;
}
- /* lookup */
- rc = getaddrinfo(saddr->host, saddr->port, &ai, &res);
-
- /* At least FreeBSD and OS-X 10.6 declare AI_V4MAPPED but
- * then don't implement it in their getaddrinfo(). Detect
- * this and retry without the flag since that's preferrable
- * to a fatal error
- */
- if (rc == EAI_BADFLAGS &&
- (ai.ai_flags & AI_V4MAPPED)) {
- atomic_set(&useV4Mapped, 0);
+ if ((check_ai_v4mapped(saddr->host, saddr->port, &ai, res)) == 0) {
ai.ai_flags &= ~AI_V4MAPPED;
- rc = getaddrinfo(saddr->host, saddr->port, &ai, &res);
}
- if (rc != 0) {
+
+ if ((rc = getaddrinfo(saddr->host, saddr->port, &ai, &res)) != 0) {
error_setg(errp, "address resolution failed for %s:%s: %s",
saddr->host, saddr->port, gai_strerror(rc));
return NULL;
@@ -469,7 +479,7 @@ static int inet_dgram_saddr(InetSocketAddress *sraddr,
/* lookup peer addr */
memset(&ai,0, sizeof(ai));
- ai.ai_flags = AI_CANONNAME | AI_V4MAPPED | AI_ADDRCONFIG;
+ ai.ai_flags = AI_CANONNAME | AI_ADDRCONFIG | AI_V4MAPPED;
ai.ai_family = inet_ai_family_from_address(sraddr, &err);
ai.ai_socktype = SOCK_DGRAM;
@@ -488,6 +498,10 @@ static int inet_dgram_saddr(InetSocketAddress *sraddr,
goto err;
}
+ if ((check_ai_v4mapped(addr, port, &ai, &peer)) == 0) {
+ ai.ai_flags &= ~AI_V4MAPPED;
+ }
+
if ((rc = getaddrinfo(addr, port, &ai, &peer)) != 0) {
error_setg(errp, "address resolution failed for %s:%s: %s",
addr, port,
gai_strerror(rc));
--
2.14.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [Qemu-devel] [PATCH] sockets: Normalize test for addrinfo flag AI_V4MAPPED
2017-11-02 16:56 [Qemu-devel] [PATCH] sockets: Normalize test for addrinfo flag AI_V4MAPPED Antonio Huete Jiménez
@ 2017-11-02 17:09 ` Daniel P. Berrange
2017-11-02 22:47 ` Antonio Huete Jiménez
0 siblings, 1 reply; 4+ messages in thread
From: Daniel P. Berrange @ 2017-11-02 17:09 UTC (permalink / raw)
To: Antonio Huete Jiménez; +Cc: qemu-devel
On Thu, Nov 02, 2017 at 04:56:29PM +0000, Antonio Huete Jiménez wrote:
> From 2b4d9d8cb617445af8f3b062f917dfea42dbdc27 Mon Sep 17 00:00:00 2001
> From: Antonio Huete Jimenez <tuxillo@quantumachine.net>
> Date: Thu, 2 Nov 2017 17:46:24 +0100
> Subject: [PATCH] sockets: Normalize test for addrinfo flag AI_V4MAPPED
Can you explain why you're making this change....
>
> Signed-off-by: Antonio Huete Jimenez <tuxillo@quantumachine.net>
> ---
> util/qemu-sockets.c | 54
> +++++++++++++++++++++++++++++++++--------------------
> 1 file changed, 34 insertions(+), 20 deletions(-)
>
> diff --git a/util/qemu-sockets.c b/util/qemu-sockets.c
> index b47fb45885..ce35b6a998 100644
> --- a/util/qemu-sockets.c
> +++ b/util/qemu-sockets.c
> @@ -44,7 +44,6 @@
> # define AI_NUMERICSERV 0
> #endif
>
> -
> static int inet_getport(struct addrinfo *e)
> {
> struct sockaddr_in *i4;
> @@ -149,6 +148,31 @@ int inet_ai_family_from_address(InetSocketAddress *addr,
> return PF_UNSPEC;
> }
>
> +static int
> +check_ai_v4mapped(const char *host, const char *port, struct addrinfo *ai,
> + struct addrinfo *res)
> +{
> + static int useV4Mapped = -1;
> + int rc;
> +
> + /* At least FreeBSD and OS-X 10.6 declare AI_V4MAPPED but
> + * then don't implement it in their getaddrinfo().
> + * Unconditionally deselect AI_V4MAPPED option upon
> + * getaddrinfo() failure, the next call to it will have to
> + * do the error handling.
> + */
> + if (atomic_read(&useV4Mapped) == -1) {
> + rc = getaddrinfo(host, port, ai, &res);
> + if (rc == 0 && (ai->ai_flags & AI_V4MAPPED)) {
> + atomic_set(&useV4Mapped, 1);
> + } else {
> + atomic_set(&useV4Mapped, 0);
> + }
> + }
> +
> + return useV4Mapped;
> +}
> +
> static int create_fast_reuse_socket(struct addrinfo *e)
> {
> int slisten = qemu_socket(e->ai_family, e->ai_socktype, e->ai_protocol);
> @@ -378,14 +402,10 @@ static struct addrinfo
> *inet_parse_connect_saddr(InetSocketAddress *saddr,
> struct addrinfo ai, *res;
> int rc;
> Error *err = NULL;
> - static int useV4Mapped = 1;
>
> memset(&ai, 0, sizeof(ai));
>
> - ai.ai_flags = AI_CANONNAME | AI_ADDRCONFIG;
> - if (atomic_read(&useV4Mapped)) {
> - ai.ai_flags |= AI_V4MAPPED;
> - }
> + ai.ai_flags = AI_CANONNAME | AI_ADDRCONFIG | AI_V4MAPPED;
> ai.ai_family = inet_ai_family_from_address(saddr, &err);
> ai.ai_socktype = SOCK_STREAM;
>
> @@ -399,21 +419,11 @@ static struct addrinfo
> *inet_parse_connect_saddr(InetSocketAddress *saddr,
> return NULL;
> }
>
> - /* lookup */
> - rc = getaddrinfo(saddr->host, saddr->port, &ai, &res);
> -
> - /* At least FreeBSD and OS-X 10.6 declare AI_V4MAPPED but
> - * then don't implement it in their getaddrinfo(). Detect
> - * this and retry without the flag since that's preferrable
> - * to a fatal error
> - */
> - if (rc == EAI_BADFLAGS &&
> - (ai.ai_flags & AI_V4MAPPED)) {
> - atomic_set(&useV4Mapped, 0);
> + if ((check_ai_v4mapped(saddr->host, saddr->port, &ai, res)) == 0) {
> ai.ai_flags &= ~AI_V4MAPPED;
> - rc = getaddrinfo(saddr->host, saddr->port, &ai, &res);
> }
> - if (rc != 0) {
> +
> + if ((rc = getaddrinfo(saddr->host, saddr->port, &ai, &res)) != 0) {
> error_setg(errp, "address resolution failed for %s:%s: %s",
> saddr->host, saddr->port, gai_strerror(rc));
> return NULL;
> @@ -469,7 +479,7 @@ static int inet_dgram_saddr(InetSocketAddress *sraddr,
>
> /* lookup peer addr */
> memset(&ai,0, sizeof(ai));
> - ai.ai_flags = AI_CANONNAME | AI_V4MAPPED | AI_ADDRCONFIG;
> + ai.ai_flags = AI_CANONNAME | AI_ADDRCONFIG | AI_V4MAPPED;
Gratuitous re-ordering of code with no functional change.
> ai.ai_family = inet_ai_family_from_address(sraddr, &err);
> ai.ai_socktype = SOCK_DGRAM;
>
> @@ -488,6 +498,10 @@ static int inet_dgram_saddr(InetSocketAddress *sraddr,
> goto err;
> }
>
> + if ((check_ai_v4mapped(addr, port, &ai, &peer)) == 0) {
> + ai.ai_flags &= ~AI_V4MAPPED;
> + }
> +
> if ((rc = getaddrinfo(addr, port, &ai, &peer)) != 0) {
> error_setg(errp, "address resolution failed for %s:%s: %s", addr,
> port,
> gai_strerror(rc));
> --
> 2.14.1
>
>
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Qemu-devel] [PATCH] sockets: Normalize test for addrinfo flag AI_V4MAPPED
2017-11-02 17:09 ` Daniel P. Berrange
@ 2017-11-02 22:47 ` Antonio Huete Jiménez
2017-11-03 14:52 ` Daniel P. Berrange
0 siblings, 1 reply; 4+ messages in thread
From: Antonio Huete Jiménez @ 2017-11-02 22:47 UTC (permalink / raw)
To: Daniel P. Berrange; +Cc: qemu-devel
"Daniel P. Berrange" <berrange@redhat.com> escribió:
> On Thu, Nov 02, 2017 at 04:56:29PM +0000, Antonio Huete Jiménez wrote:
>> From 2b4d9d8cb617445af8f3b062f917dfea42dbdc27 Mon Sep 17 00:00:00 2001
>> From: Antonio Huete Jimenez <tuxillo@quantumachine.net>
>> Date: Thu, 2 Nov 2017 17:46:24 +0100
>> Subject: [PATCH] sockets: Normalize test for addrinfo flag AI_V4MAPPED
>
> Can you explain why you're making this change....
Hi Daniel,
Thanks for reviewing the patch. I am trying to fix 'make check' for
DragonFly BSD and this is the first issue I encountered.
With help from people in #qemu we found that it may be necessary to
consistently check for the availability of this flag so I thought it
would be a good idea to do it like this.
>
>>
>> Signed-off-by: Antonio Huete Jimenez <tuxillo@quantumachine.net>
>> ---
>> util/qemu-sockets.c | 54
>> +++++++++++++++++++++++++++++++++--------------------
>> 1 file changed, 34 insertions(+), 20 deletions(-)
>>
>> diff --git a/util/qemu-sockets.c b/util/qemu-sockets.c
>> index b47fb45885..ce35b6a998 100644
>> --- a/util/qemu-sockets.c
>> +++ b/util/qemu-sockets.c
>> @@ -44,7 +44,6 @@
>> # define AI_NUMERICSERV 0
>> #endif
>>
>> -
>> static int inet_getport(struct addrinfo *e)
>> {
>> struct sockaddr_in *i4;
>> @@ -149,6 +148,31 @@ int
>> inet_ai_family_from_address(InetSocketAddress *addr,
>> return PF_UNSPEC;
>> }
>>
>> +static int
>> +check_ai_v4mapped(const char *host, const char *port, struct addrinfo *ai,
>> + struct addrinfo *res)
>> +{
>> + static int useV4Mapped = -1;
>> + int rc;
>> +
>> + /* At least FreeBSD and OS-X 10.6 declare AI_V4MAPPED but
>> + * then don't implement it in their getaddrinfo().
>> + * Unconditionally deselect AI_V4MAPPED option upon
>> + * getaddrinfo() failure, the next call to it will have to
>> + * do the error handling.
>> + */
>> + if (atomic_read(&useV4Mapped) == -1) {
>> + rc = getaddrinfo(host, port, ai, &res);
>> + if (rc == 0 && (ai->ai_flags & AI_V4MAPPED)) {
>> + atomic_set(&useV4Mapped, 1);
>> + } else {
>> + atomic_set(&useV4Mapped, 0);
>> + }
>> + }
>> +
>> + return useV4Mapped;
>> +}
>> +
>> static int create_fast_reuse_socket(struct addrinfo *e)
>> {
>> int slisten = qemu_socket(e->ai_family, e->ai_socktype,
>> e->ai_protocol);
>> @@ -378,14 +402,10 @@ static struct addrinfo
>> *inet_parse_connect_saddr(InetSocketAddress *saddr,
>> struct addrinfo ai, *res;
>> int rc;
>> Error *err = NULL;
>> - static int useV4Mapped = 1;
>>
>> memset(&ai, 0, sizeof(ai));
>>
>> - ai.ai_flags = AI_CANONNAME | AI_ADDRCONFIG;
>> - if (atomic_read(&useV4Mapped)) {
>> - ai.ai_flags |= AI_V4MAPPED;
>> - }
>> + ai.ai_flags = AI_CANONNAME | AI_ADDRCONFIG | AI_V4MAPPED;
>> ai.ai_family = inet_ai_family_from_address(saddr, &err);
>> ai.ai_socktype = SOCK_STREAM;
>>
>> @@ -399,21 +419,11 @@ static struct addrinfo
>> *inet_parse_connect_saddr(InetSocketAddress *saddr,
>> return NULL;
>> }
>>
>> - /* lookup */
>> - rc = getaddrinfo(saddr->host, saddr->port, &ai, &res);
>> -
>> - /* At least FreeBSD and OS-X 10.6 declare AI_V4MAPPED but
>> - * then don't implement it in their getaddrinfo(). Detect
>> - * this and retry without the flag since that's preferrable
>> - * to a fatal error
>> - */
>> - if (rc == EAI_BADFLAGS &&
>> - (ai.ai_flags & AI_V4MAPPED)) {
>> - atomic_set(&useV4Mapped, 0);
>> + if ((check_ai_v4mapped(saddr->host, saddr->port, &ai, res)) == 0) {
>> ai.ai_flags &= ~AI_V4MAPPED;
>> - rc = getaddrinfo(saddr->host, saddr->port, &ai, &res);
>> }
>> - if (rc != 0) {
>> +
>> + if ((rc = getaddrinfo(saddr->host, saddr->port, &ai, &res)) != 0) {
>> error_setg(errp, "address resolution failed for %s:%s: %s",
>> saddr->host, saddr->port, gai_strerror(rc));
>> return NULL;
>> @@ -469,7 +479,7 @@ static int inet_dgram_saddr(InetSocketAddress *sraddr,
>>
>> /* lookup peer addr */
>> memset(&ai,0, sizeof(ai));
>> - ai.ai_flags = AI_CANONNAME | AI_V4MAPPED | AI_ADDRCONFIG;
>> + ai.ai_flags = AI_CANONNAME | AI_ADDRCONFIG | AI_V4MAPPED;
>
> Gratuitous re-ordering of code with no functional change.
Agree, can change it.
>
>> ai.ai_family = inet_ai_family_from_address(sraddr, &err);
>> ai.ai_socktype = SOCK_DGRAM;
>>
>> @@ -488,6 +498,10 @@ static int inet_dgram_saddr(InetSocketAddress *sraddr,
>> goto err;
>> }
>>
>> + if ((check_ai_v4mapped(addr, port, &ai, &peer)) == 0) {
>> + ai.ai_flags &= ~AI_V4MAPPED;
>> + }
>> +
>> if ((rc = getaddrinfo(addr, port, &ai, &peer)) != 0) {
>> error_setg(errp, "address resolution failed for %s:%s: %s", addr,
>> port,
>> gai_strerror(rc));
>> --
>> 2.14.1
>>
>>
>
> Regards,
> Daniel
> --
> |: https://berrange.com -o-
> https://www.flickr.com/photos/dberrange :|
> |: https://libvirt.org -o-
> https://fstop138.berrange.com :|
> |: https://entangle-photo.org -o-
> https://www.instagram.com/dberrange :|
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Qemu-devel] [PATCH] sockets: Normalize test for addrinfo flag AI_V4MAPPED
2017-11-02 22:47 ` Antonio Huete Jiménez
@ 2017-11-03 14:52 ` Daniel P. Berrange
0 siblings, 0 replies; 4+ messages in thread
From: Daniel P. Berrange @ 2017-11-03 14:52 UTC (permalink / raw)
To: Antonio Huete Jiménez; +Cc: qemu-devel
On Thu, Nov 02, 2017 at 10:47:10PM +0000, Antonio Huete Jiménez wrote:
>
> "Daniel P. Berrange" <berrange@redhat.com> escribió:
>
> > On Thu, Nov 02, 2017 at 04:56:29PM +0000, Antonio Huete Jiménez wrote:
> > > From 2b4d9d8cb617445af8f3b062f917dfea42dbdc27 Mon Sep 17 00:00:00 2001
> > > From: Antonio Huete Jimenez <tuxillo@quantumachine.net>
> > > Date: Thu, 2 Nov 2017 17:46:24 +0100
> > > Subject: [PATCH] sockets: Normalize test for addrinfo flag AI_V4MAPPED
> >
> > Can you explain why you're making this change....
>
> Hi Daniel,
>
> Thanks for reviewing the patch. I am trying to fix 'make check' for
> DragonFly BSD and this is the first issue I encountered.
Ok, it would be helpful if the commit msg could mention which test fails
without this fix, ideally with the actual error message (if one is given,
as opposed to an abort()).
> > > Signed-off-by: Antonio Huete Jimenez <tuxillo@quantumachine.net>
> > > ---
> > > util/qemu-sockets.c | 54
> > > +++++++++++++++++++++++++++++++++--------------------
> > > 1 file changed, 34 insertions(+), 20 deletions(-)
> > >
> > > diff --git a/util/qemu-sockets.c b/util/qemu-sockets.c
> > > index b47fb45885..ce35b6a998 100644
> > > --- a/util/qemu-sockets.c
> > > +++ b/util/qemu-sockets.c
> > > @@ -44,7 +44,6 @@
> > > # define AI_NUMERICSERV 0
> > > #endif
> > >
> > > -
> > > static int inet_getport(struct addrinfo *e)
> > > {
> > > struct sockaddr_in *i4;
> > > @@ -149,6 +148,31 @@ int
> > > inet_ai_family_from_address(InetSocketAddress *addr,
> > > return PF_UNSPEC;
> > > }
> > >
> > > +static int
> > > +check_ai_v4mapped(const char *host, const char *port, struct addrinfo *ai,
> > > + struct addrinfo *res)
> > > +{
> > > + static int useV4Mapped = -1;
> > > + int rc;
> > > +
> > > + /* At least FreeBSD and OS-X 10.6 declare AI_V4MAPPED but
> > > + * then don't implement it in their getaddrinfo().
> > > + * Unconditionally deselect AI_V4MAPPED option upon
> > > + * getaddrinfo() failure, the next call to it will have to
> > > + * do the error handling.
> > > + */
> > > + if (atomic_read(&useV4Mapped) == -1) {
> > > + rc = getaddrinfo(host, port, ai, &res);
> > > + if (rc == 0 && (ai->ai_flags & AI_V4MAPPED)) {
> > > + atomic_set(&useV4Mapped, 1);
> > > + } else {
> > > + atomic_set(&useV4Mapped, 0);
This is treating every single failure of getaddrinfo as a sign to turn
off AI_V4MAPPED, which is really bad. I'm not a huge fan of doing the
extra getaddrinfo() call, even though it is protected behind the atomic
var. I'd prefer to just keep current style of checking the result after
the main call to getaddrinfo() - just extend that to the dgram case too.
A single 'static int useV4Mapped = 1' could still be used in the global
namespace though.
> > > + }
> > > + }
> > > +
> > > + return useV4Mapped;
> > > +}
> > > +
> > > static int create_fast_reuse_socket(struct addrinfo *e)
> > > {
> > > int slisten = qemu_socket(e->ai_family, e->ai_socktype,
> > > e->ai_protocol);
> > > @@ -378,14 +402,10 @@ static struct addrinfo
> > > *inet_parse_connect_saddr(InetSocketAddress *saddr,
> > > struct addrinfo ai, *res;
> > > int rc;
> > > Error *err = NULL;
> > > - static int useV4Mapped = 1;
> > >
> > > memset(&ai, 0, sizeof(ai));
> > >
> > > - ai.ai_flags = AI_CANONNAME | AI_ADDRCONFIG;
> > > - if (atomic_read(&useV4Mapped)) {
> > > - ai.ai_flags |= AI_V4MAPPED;
> > > - }
> > > + ai.ai_flags = AI_CANONNAME | AI_ADDRCONFIG | AI_V4MAPPED;
> > > ai.ai_family = inet_ai_family_from_address(saddr, &err);
> > > ai.ai_socktype = SOCK_STREAM;
> > >
> > > @@ -399,21 +419,11 @@ static struct addrinfo
> > > *inet_parse_connect_saddr(InetSocketAddress *saddr,
> > > return NULL;
> > > }
> > >
> > > - /* lookup */
> > > - rc = getaddrinfo(saddr->host, saddr->port, &ai, &res);
> > > -
> > > - /* At least FreeBSD and OS-X 10.6 declare AI_V4MAPPED but
> > > - * then don't implement it in their getaddrinfo(). Detect
> > > - * this and retry without the flag since that's preferrable
> > > - * to a fatal error
> > > - */
> > > - if (rc == EAI_BADFLAGS &&
> > > - (ai.ai_flags & AI_V4MAPPED)) {
> > > - atomic_set(&useV4Mapped, 0);
> > > + if ((check_ai_v4mapped(saddr->host, saddr->port, &ai, res)) == 0) {
> > > ai.ai_flags &= ~AI_V4MAPPED;
> > > - rc = getaddrinfo(saddr->host, saddr->port, &ai, &res);
> > > }
> > > - if (rc != 0) {
> > > +
> > > + if ((rc = getaddrinfo(saddr->host, saddr->port, &ai, &res)) != 0) {
> > > error_setg(errp, "address resolution failed for %s:%s: %s",
> > > saddr->host, saddr->port, gai_strerror(rc));
> > > return NULL;
> > > @@ -469,7 +479,7 @@ static int inet_dgram_saddr(InetSocketAddress *sraddr,
> > >
> > > /* lookup peer addr */
> > > memset(&ai,0, sizeof(ai));
> > > - ai.ai_flags = AI_CANONNAME | AI_V4MAPPED | AI_ADDRCONFIG;
> > > + ai.ai_flags = AI_CANONNAME | AI_ADDRCONFIG | AI_V4MAPPED;
> >
> > Gratuitous re-ordering of code with no functional change.
>
> Agree, can change it.
>
> >
> > > ai.ai_family = inet_ai_family_from_address(sraddr, &err);
> > > ai.ai_socktype = SOCK_DGRAM;
> > >
> > > @@ -488,6 +498,10 @@ static int inet_dgram_saddr(InetSocketAddress *sraddr,
> > > goto err;
> > > }
> > >
> > > + if ((check_ai_v4mapped(addr, port, &ai, &peer)) == 0) {
> > > + ai.ai_flags &= ~AI_V4MAPPED;
> > > + }
> > > +
> > > if ((rc = getaddrinfo(addr, port, &ai, &peer)) != 0) {
> > > error_setg(errp, "address resolution failed for %s:%s: %s", addr,
> > > port,
> > > gai_strerror(rc));
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-11-03 14:52 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-02 16:56 [Qemu-devel] [PATCH] sockets: Normalize test for addrinfo flag AI_V4MAPPED Antonio Huete Jiménez
2017-11-02 17:09 ` Daniel P. Berrange
2017-11-02 22:47 ` Antonio Huete Jiménez
2017-11-03 14:52 ` Daniel P. Berrange
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.