summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorUlrich Drepper <drepper@redhat.com>2007-10-14 17:58:03 +0000
committerUlrich Drepper <drepper@redhat.com>2007-10-14 17:58:03 +0000
commit6f55ed8885ed1cb45854067743590c44698a1fe1 (patch)
tree8c9e256993fce19b5433b93d73ea0852472af3f7
parentfa76dde2cf59865481793b9f233df74aa5c7d4c1 (diff)
* sysdeps/posix/getaddrinfo.c (struct sort_result): Addcvs/fedora-glibc-20071014T1847
service_order. (rfc3484_sort): Make sure that even of qsort doesn't support stable sorting it is stable by comparing service_order. (getaddrinfo): Initialize service_order.
-rw-r--r--ChangeLog6
-rw-r--r--sysdeps/posix/getaddrinfo.c9
2 files changed, 13 insertions, 2 deletions
diff --git a/ChangeLog b/ChangeLog
index c16c6ed0fa..16d3a2e3b4 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,11 @@
2007-10-14 Ulrich Drepper <drepper@redhat.com>
+ * sysdeps/posix/getaddrinfo.c (struct sort_result): Add
+ service_order.
+ (rfc3484_sort): Make sure that even of qsort doesn't support
+ stable sorting it is stable by comparing service_order.
+ (getaddrinfo): Initialize service_order.
+
* include/time.h: Declare __tzset_parse_tz and __tzset_compute.
* time/tzset.c (tzset_internal): Break TZ string parsing out into
__tzset_parse_tz and updating of daylight, timezone, tzname into
diff --git a/sysdeps/posix/getaddrinfo.c b/sysdeps/posix/getaddrinfo.c
index 221b41dd00..8f37ec5c11 100644
--- a/sysdeps/posix/getaddrinfo.c
+++ b/sysdeps/posix/getaddrinfo.c
@@ -1002,6 +1002,7 @@ struct sort_result
{
struct addrinfo *dest_addr;
struct sockaddr_storage source_addr;
+ size_t service_order;
uint8_t source_addr_len;
bool got_source_addr;
uint8_t source_addr_flags;
@@ -1403,8 +1404,11 @@ rfc3484_sort (const void *p1, const void *p2)
}
- /* Rule 10: Otherwise, leave the order unchanged. */
- return 0;
+ /* Rule 10: Otherwise, leave the order unchanged. To ensure this
+ compare with the value indicating the order in which the entries
+ have been received from the services. NB: no two entries can have
+ the same order so the test will never return zero. */
+ return a1->service_order < a2->service_order ? -1 : 1;
}
@@ -1951,6 +1955,7 @@ getaddrinfo (const char *name, const char *service,
{
results[i].dest_addr = q;
results[i].got_source_addr = false;
+ results[i].service_order = i;
/* If we just looked up the address for a different
protocol, reuse the result. */