diff options
| author | Ulrich Drepper <drepper@redhat.com> | 2010-03-26 05:40:27 -0700 |
|---|---|---|
| committer | Ulrich Drepper <drepper@redhat.com> | 2010-03-26 05:40:27 -0700 |
| commit | 7c9302908ee9ce1c21d930923a879c2cbef5242f (patch) | |
| tree | 6512a54a64f265695705eaf79329fa2e0ad3f770 | |
| parent | c3dfadb87e16f0fab6f4d5242bcecf06c02976c7 (diff) | |
| download | glibc-7c9302908ee9ce1c21d930923a879c2cbef5242f.zip glibc-7c9302908ee9ce1c21d930923a879c2cbef5242f.tar.gz glibc-7c9302908ee9ce1c21d930923a879c2cbef5242f.tar.bz2 | |
Don't abort immediately on successful lookup in getaddrinfo.
When not using gethostbyname4 methods we immediately aborted the loop
over the nss modules on the first successful lookup. While this is
almost always what is wanted the nsswitch.conf file allows to select
something different.
| -rw-r--r-- | ChangeLog | 5 | ||||
| -rw-r--r-- | sysdeps/posix/getaddrinfo.c | 20 |
2 files changed, 15 insertions, 10 deletions
@@ -1,3 +1,8 @@ +2010-03-26 Ulrich Drepper <drepper@redhat.com> + + * sysdeps/posix/getaddrinfo.c (gaih_inet): Don't assume success always + means aborting the loop over the NSS methods. + 2010-03-25 Andreas Schwab <schwab@redhat.com> * sysdeps/posix/getaddrinfo.c (gaih_inet): Reset no_data before diff --git a/sysdeps/posix/getaddrinfo.c b/sysdeps/posix/getaddrinfo.c index 8b7e38f..597189f 100644 --- a/sysdeps/posix/getaddrinfo.c +++ b/sysdeps/posix/getaddrinfo.c @@ -817,17 +817,17 @@ gaih_inet (const char *name, const struct gaih_service *service, canon = name; } } - - break; } - - /* We can have different states for AF_INET and - AF_INET6. Try to find a useful one for both. */ - if (inet6_status == NSS_STATUS_TRYAGAIN) - status = NSS_STATUS_TRYAGAIN; - else if (status == NSS_STATUS_UNAVAIL - && inet6_status != NSS_STATUS_UNAVAIL) - status = inet6_status; + else + { + /* We can have different states for AF_INET and + AF_INET6. Try to find a useful one for both. */ + if (inet6_status == NSS_STATUS_TRYAGAIN) + status = NSS_STATUS_TRYAGAIN; + else if (status == NSS_STATUS_UNAVAIL + && inet6_status != NSS_STATUS_UNAVAIL) + status = inet6_status; + } } else status = NSS_STATUS_UNAVAIL; |