Return EAI_AGAIN for AF_UNSPEC when herrno is TRY_AGAIN (BZ #16849)

  getaddrinfo correctly returns EAI_AGAIN for AF_INET and AF_INET6
queries.  For AF_UNSPEC however, an older change
(a682a1bf553b1efe4dbb03207fece5b719cec482) broke the check and due to
that the returned error was EAI_NONAME.

This patch fixes the check so that a non-authoritative not-found is
returned as EAI_AGAIN to the user instead of EAI_NONAME.

Tested on x86_64.

Siddhesh

	[BZ #16849]
	* sysdeps/posix/getaddrinfo.c (gaih_inet): Only check for
	herrno to return EAI_AGAIN.

---
 sysdeps/posix/getaddrinfo.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
  

Ping!

On Wed, Apr 16, 2014 at 07:38:24PM +0530, Siddhesh Poyarekar wrote:
> getaddrinfo correctly returns EAI_AGAIN for AF_INET and AF_INET6
> queries.  For AF_UNSPEC however, an older change
> (a682a1bf553b1efe4dbb03207fece5b719cec482) broke the check and due to
> that the returned error was EAI_NONAME.
> 
> This patch fixes the check so that a non-authoritative not-found is
> returned as EAI_AGAIN to the user instead of EAI_NONAME.
> 
> Tested on x86_64.
> 
> Siddhesh
> 
> 	[BZ #16849]
> 	* sysdeps/posix/getaddrinfo.c (gaih_inet): Only check for
> 	herrno to return EAI_AGAIN.
> 
> ---
>  sysdeps/posix/getaddrinfo.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/sysdeps/posix/getaddrinfo.c b/sysdeps/posix/getaddrinfo.c
> index 3385bed..f62df19 100644
> --- a/sysdeps/posix/getaddrinfo.c
> +++ b/sysdeps/posix/getaddrinfo.c
> @@ -863,8 +863,7 @@ gaih_inet (const char *name, const struct gaih_service *service,
>  		      if (status != NSS_STATUS_TRYAGAIN
>  			  || rc != ERANGE || herrno != NETDB_INTERNAL)
>  			{
> -			  if (status == NSS_STATUS_TRYAGAIN
> -			      && herrno == TRY_AGAIN)
> +			  if (herrno == TRY_AGAIN)
>  			    no_data = EAI_AGAIN;
>  			  else
>  			    no_data = herrno == NO_DATA;
> -- 
> 1.8.3.1
>
  

Ping!

On Tue, Apr 22, 2014 at 03:16:57PM +0530, Siddhesh Poyarekar wrote:
> Ping!
> 
> On Wed, Apr 16, 2014 at 07:38:24PM +0530, Siddhesh Poyarekar wrote:
> > getaddrinfo correctly returns EAI_AGAIN for AF_INET and AF_INET6
> > queries.  For AF_UNSPEC however, an older change
> > (a682a1bf553b1efe4dbb03207fece5b719cec482) broke the check and due to
> > that the returned error was EAI_NONAME.
> > 
> > This patch fixes the check so that a non-authoritative not-found is
> > returned as EAI_AGAIN to the user instead of EAI_NONAME.
> > 
> > Tested on x86_64.
> > 
> > Siddhesh
> > 
> > 	[BZ #16849]
> > 	* sysdeps/posix/getaddrinfo.c (gaih_inet): Only check for
> > 	herrno to return EAI_AGAIN.
> > 
> > ---
> >  sysdeps/posix/getaddrinfo.c | 3 +--
> >  1 file changed, 1 insertion(+), 2 deletions(-)
> > 
> > diff --git a/sysdeps/posix/getaddrinfo.c b/sysdeps/posix/getaddrinfo.c
> > index 3385bed..f62df19 100644
> > --- a/sysdeps/posix/getaddrinfo.c
> > +++ b/sysdeps/posix/getaddrinfo.c
> > @@ -863,8 +863,7 @@ gaih_inet (const char *name, const struct gaih_service *service,
> >  		      if (status != NSS_STATUS_TRYAGAIN
> >  			  || rc != ERANGE || herrno != NETDB_INTERNAL)
> >  			{
> > -			  if (status == NSS_STATUS_TRYAGAIN
> > -			      && herrno == TRY_AGAIN)
> > +			  if (herrno == TRY_AGAIN)
> >  			    no_data = EAI_AGAIN;
> >  			  else
> >  			    no_data = herrno == NO_DATA;
> > -- 
> > 1.8.3.1
> > 
> 
>
  

On Wed, Apr 16, 2014 at 07:38:24PM +0530, Siddhesh Poyarekar wrote:
> getaddrinfo correctly returns EAI_AGAIN for AF_INET and AF_INET6
> queries.  For AF_UNSPEC however, an older change
> (a682a1bf553b1efe4dbb03207fece5b719cec482) broke the check and due to
> that the returned error was EAI_NONAME.
> 
> This patch fixes the check so that a non-authoritative not-found is
> returned as EAI_AGAIN to the user instead of EAI_NONAME.
> 
> Tested on x86_64.
> 
looks ok for me