Hello,

I am using the curl-multi-API on windows to do a lot of http-requests to
a lot of different hosts. All hosts are identified directly via IPv4-addresses
instead of DNS-entries. Unfortunately curl fires up a new thread to simulate
async-DNS-resolve. In my case that should not be necessary, because I already
provide the dotted IPv4-address. (I 'm not using c-ares).
That thread-start costs a lot of CPU-time for me and can be patched easily.

As I looked at the source this happens in asyn-thread.c in function
Curl_resolver_getaddrinfo.
I don't see any reason against calling Curl_inet_pton before starting a new
thread to check if we already have a dotted IP. Thats the same what already
happens in the asyn-ares.c solution. I have a patch against the nightly build
of august 2nd attached. I am not used to IPv6 and hope the patch didn't break
anything with IPv6.

Regards,

Christian Hägele

--- curl-7.21.8-20110802\lib\asyn-thread.c Wed Jul 27 04:00:16 2011
+++ curl-7.21.8-20110802_patched\lib\asyn-thread.c Tue Aug 02 14:44:45 2011
@@ -626,14 +626,28 @@
                                          int *waitp)
 {
   struct addrinfo hints;
+ struct in_addr in;
   Curl_addrinfo *res;
   int error;
   char sbuf[NI_MAXSERV];
   int pf = PF_INET;
+#ifdef ENABLE_IPV6 /* CURLRES_IPV6 */
+ struct in6_addr in6;
+#endif /* CURLRES_IPV6 */
 
   *waitp = 0; /* default to synchronous response */
 
+ /* First check if this is an IPv4 address string */
+ if(Curl_inet_pton(AF_INET, hostname, &in) > 0)
+ /* This is a dotted IP address 123.123.123.123-style */
+ return Curl_ip2addr(AF_INET, &in, hostname, port);
+
 #ifndef CURLRES_IPV4
+ /* Otherwise, check if this is an IPv6 address string */
+ if(Curl_inet_pton (AF_INET6, hostname, &in6) > 0)
+ /* This must be an IPv6 address literal. */
+ return Curl_ip2addr(AF_INET6, &in6, hostname, port);
+
   /*
    * Check if a limited name resolve has been requested.
    */

-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette: http://curl.haxx.se/mail/etiquette.html
Received on 2011-08-02