From d118312922cc5415b26afd2eae2849c2c56a3359 Mon Sep 17 00:00:00 2001 From: Gisle Vanem Date: Tue, 8 Feb 2005 12:36:13 +0000 Subject: Curl_addrinfo?_callback() and addrinfo_callback() now returns CURLE_OK or CURLE_OUT_OF_MEMORY. Add typecast in hostares.c. --- lib/hostares.c | 2 +- lib/hostasyn.c | 30 ++++++++++++++++++------------ lib/hostip.h | 12 ++++++------ lib/hostthre.c | 26 +++++++++++++------------- 4 files changed, 38 insertions(+), 32 deletions(-) diff --git a/lib/hostares.c b/lib/hostares.c index a54e25eda..452c7588c 100644 --- a/lib/hostares.c +++ b/lib/hostares.c @@ -291,7 +291,7 @@ Curl_addrinfo *Curl_getaddrinfo(struct connectdata *conn, /* areschannel is already setup in the Curl_open() function */ ares_gethostbyname(data->state.areschannel, hostname, PF_INET, - Curl_addrinfo4_callback, conn); + (ares_callback)Curl_addrinfo4_callback, conn); *waitp = TRUE; /* please wait for the response */ } diff --git a/lib/hostasyn.c b/lib/hostasyn.c index 2abad14fd..ec7a34a43 100644 --- a/lib/hostasyn.c +++ b/lib/hostasyn.c @@ -108,12 +108,13 @@ * * The storage operation locks and unlocks the DNS cache. */ -static void addrinfo_callback(void *arg, /* "struct connectdata *" */ - int status, - void *addr) +static CURLcode addrinfo_callback(void *arg, /* "struct connectdata *" */ + int status, + void *addr) { struct connectdata *conn = (struct connectdata *)arg; struct Curl_dns_entry *dns = NULL; + CURLcode rc = CURLE_OK; conn->async.status = status; @@ -135,13 +136,17 @@ static void addrinfo_callback(void *arg, /* "struct connectdata *" */ dns = Curl_cache_addr(data, ai, conn->async.hostname, conn->async.port); - if(!dns) + if(!dns) { /* failed to store, cleanup and return error */ Curl_freeaddrinfo(ai); + rc = CURLE_OUT_OF_MEMORY; + } if(data->share) Curl_share_unlock(data, CURL_LOCK_DATA_DNS); } + else + rc = CURLE_OUT_OF_MEMORY; } conn->async.dns = dns; @@ -153,21 +158,22 @@ static void addrinfo_callback(void *arg, /* "struct connectdata *" */ /* ipv4: The input hostent struct will be freed by ares when we return from this function */ + return rc; } -void Curl_addrinfo4_callback(void *arg, /* "struct connectdata *" */ - int status, - struct hostent *hostent) +CURLcode Curl_addrinfo4_callback(void *arg, /* "struct connectdata *" */ + int status, + struct hostent *hostent) { - addrinfo_callback(arg, status, hostent); + return addrinfo_callback(arg, status, hostent); } #ifdef CURLRES_IPV6 -void Curl_addrinfo6_callback(void *arg, /* "struct connectdata *" */ - int status, - struct addrinfo *ai) +CURLcode Curl_addrinfo6_callback(void *arg, /* "struct connectdata *" */ + int status, + struct addrinfo *ai) { - addrinfo_callback(arg, status, ai); + return addrinfo_callback(arg, status, ai); } #endif diff --git a/lib/hostip.h b/lib/hostip.h index 75ad10b90..4fd03c1af 100644 --- a/lib/hostip.h +++ b/lib/hostip.h @@ -199,14 +199,14 @@ int curl_dogetnameinfo(const struct sockaddr *sa, socklen_t salen, /* This is the callback function that is used when we build with asynch resolve, ipv4 */ -void Curl_addrinfo4_callback(void *arg, - int status, - struct hostent *hostent); +CURLcode Curl_addrinfo4_callback(void *arg, + int status, + struct hostent *hostent); /* This is the callback function that is used when we build with asynch resolve, ipv6 */ -void Curl_addrinfo6_callback(void *arg, - int status, - struct addrinfo *ai); +CURLcode Curl_addrinfo6_callback(void *arg, + int status, + struct addrinfo *ai); /* [ipv4 only] Creates a Curl_addrinfo struct from a numerical-only IP diff --git a/lib/hostthre.c b/lib/hostthre.c index 01625e6a9..47c9ea1c8 100644 --- a/lib/hostthre.c +++ b/lib/hostthre.c @@ -190,7 +190,7 @@ static unsigned __stdcall gethostbyname_thread (void *arg) curr_proc, &mutex_waiting, 0, FALSE, DUPLICATE_SAME_ACCESS)) { /* failed to duplicate the mutex, no point in continuing */ - return 0; + return -1; } /* Sharing the same _iob[] element with our parent thread should @@ -214,12 +214,10 @@ static unsigned __stdcall gethostbyname_thread (void *arg) SetEvent(td->event_resolved); if (he) { - Curl_addrinfo4_callback(conn, CURL_ASYNC_SUCCESS, he); - rc = 1; + rc = Curl_addrinfo4_callback(conn, CURL_ASYNC_SUCCESS, he); } else { - Curl_addrinfo4_callback(conn, (int)WSAGetLastError(), NULL); - rc = 0; + rc = Curl_addrinfo4_callback(conn, (int)WSAGetLastError(), NULL); } TRACE(("Winsock-error %d, addr %s\n", conn->async.status, he ? inet_ntoa(*(struct in_addr*)he->h_addr) : "unknown")); @@ -260,7 +258,7 @@ static unsigned __stdcall getaddrinfo_thread (void *arg) curr_proc, &mutex_waiting, 0, FALSE, DUPLICATE_SAME_ACCESS)) { /* failed to duplicate the mutex, no point in continuing */ - return 0; + return -1; } #ifndef _WIN32_WCE @@ -286,10 +284,10 @@ static unsigned __stdcall getaddrinfo_thread (void *arg) #ifdef DEBUG_THREADING_GETADDRINFO dump_addrinfo (conn, res); #endif - Curl_addrinfo6_callback(conn, CURL_ASYNC_SUCCESS, res); + rc = Curl_addrinfo6_callback(conn, CURL_ASYNC_SUCCESS, res); } else { - Curl_addrinfo6_callback(conn, (int)WSAGetLastError(), NULL); + rc = Curl_addrinfo6_callback(conn, (int)WSAGetLastError(), NULL); TRACE(("Winsock-error %d, no address\n", conn->async.status)); } } @@ -493,7 +491,11 @@ CURLcode Curl_wait_for_resolv(struct connectdata *conn, if (!conn->async.dns) { /* a name was not resolved */ - if (td->thread_status == (DWORD)-1 || conn->async.status == NO_DATA) { + if (td->thread_status == CURLE_OUT_OF_MEMORY) { + rc = CURLE_OUT_OF_MEMORY; + failf(data, "Could not resolve host: %s", curl_easy_strerror(rc)); + } + else if (td->thread_status == (DWORD)-1 || conn->async.status == NO_DATA) { failf(data, "Resolving host timed out: %s", conn->host.name); rc = CURLE_OPERATION_TIMEDOUT; } @@ -508,10 +510,8 @@ CURLcode Curl_wait_for_resolv(struct connectdata *conn, destroy_thread_data(&conn->async); - if(CURLE_OK != rc) - /* close the connection, since we must not return failure from here - without cleaning up this connection properly */ - Curl_disconnect(conn); + if(!conn->async.dns) + conn->bits.close = TRUE; return (rc); } -- cgit v1.2.3