From 8ab137b2bc9630ce20f45fd09b4ed2149afa45fd Mon Sep 17 00:00:00 2001 From: Tommie Gannert Date: Sat, 18 Dec 2010 22:31:39 +0100 Subject: ares: ask for both IPv4 and IPv6 addresses Make the c-ares resolver code ask for both IPv4 and IPv6 addresses when IPv6 is enabled. This is a workaround for the missing ares_getaddrinfo() and is a lot easier to implement. Note that as long as c-ares returns IPv4 addresses when IPv6 addresses were requested but missing, this will cause a host's IPv4 addresses to occur twice in the DNS cache. URL: http://curl.haxx.se/mail/lib-2010-12/0041.html --- lib/hostasyn.c | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++-- 1 file changed, 55 insertions(+), 2 deletions(-) (limited to 'lib/hostasyn.c') diff --git a/lib/hostasyn.c b/lib/hostasyn.c index 127b8d332..7d35fa0e7 100644 --- a/lib/hostasyn.c +++ b/lib/hostasyn.c @@ -5,7 +5,7 @@ * | (__| |_| | _ <| |___ * \___|\___/|_| \_\_____| * - * Copyright (C) 1998 - 2009, Daniel Stenberg, , et al. + * Copyright (C) 1998 - 2010, Daniel Stenberg, , et al. * * This software is licensed as described in the file COPYING, which * you should have received as part of this distribution. The terms @@ -95,6 +95,20 @@ CURLcode Curl_addrinfo_callback(struct connectdata * conn, if(ai) { struct SessionHandle *data = conn->data; +#if defined(ENABLE_IPV6) && defined(CURLRES_ARES) /* CURLRES_IPV6 */ + Curl_addrinfo *ai_tail = ai; + + while (ai_tail->ai_next) + ai_tail = ai_tail->ai_next; + + /* Add the new results to the list of old results. */ + ai_tail->ai_next = conn->async.temp_ai; + conn->async.temp_ai = ai; + + if(--conn->async.num_pending > 0) + /* We are not done yet. Just return. */ + return CURLE_OK; +#endif if(data->share) Curl_share_lock(data, CURL_LOCK_DATA_DNS, CURL_LOCK_ACCESS_SINGLE); @@ -110,9 +124,48 @@ CURLcode Curl_addrinfo_callback(struct connectdata * conn, if(data->share) Curl_share_unlock(data, CURL_LOCK_DATA_DNS); } - else + else { +#if defined(ENABLE_IPV6) && defined(CURLRES_ARES) /* CURLRES_IPV6 */ + if(--conn->async.num_pending > 0) { + /* We are not done yet. Clean up and return. + This function will be called again. */ + if(conn->async.temp_ai) + Curl_freeaddrinfo(conn->async.temp_ai); + return CURLE_OUT_OF_MEMORY; + } +#endif rc = CURLE_OUT_OF_MEMORY; + } } +#if defined(ENABLE_IPV6) && defined(CURLRES_ARES) /* CURLRES_IPV6 */ + else + { + if(--conn->async.num_pending > 0) + /* We are not done yet. Just return. */ + return CURLE_OK; + + if(conn->async.temp_ai) { + /* We are done, and while this latest request + failed, some previous results exist. */ + struct SessionHandle *data = conn->data; + + if(data->share) + Curl_share_lock(data, CURL_LOCK_DATA_DNS, CURL_LOCK_ACCESS_SINGLE); + + dns = Curl_cache_addr(data, conn->async.temp_ai, + conn->async.hostname, + conn->async.port); + if(!dns) { + /* failed to store, cleanup and return error */ + Curl_freeaddrinfo(conn->async.temp_ai); + rc = CURLE_OUT_OF_MEMORY; + } + + if(data->share) + Curl_share_unlock(data, CURL_LOCK_DATA_DNS); + } + } +#endif conn->async.dns = dns; -- cgit v1.2.3