From: Yang Tse <yangsita@gmail.com>
Date: Thu, 2 Oct 2008 22:58:09 +0000 (+0000)
Subject: take 2 at fixing compiler warning: argument might be clobbered by `longjmp' or `vfork'
X-Git-Tag: curl-7_19_1~236
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=bdf17244128fc1c8c5b47273b1f3a6148015b079;p=curl

take 2 at fixing compiler warning: argument might be clobbered by `longjmp' or `vfork'
---

diff --git a/lib/hostip.c b/lib/hostip.c
index f8c2be63f..1a5711f55 100644
--- a/lib/hostip.c
+++ b/lib/hostip.c
@@ -524,7 +524,7 @@ int Curl_resolv_timeout(struct connectdata *conn,
                         const char *hostname,
                         int port,
                         struct Curl_dns_entry **entry,
-                        volatile long timeout)
+                        long timeoutms)
 {
 #ifdef USE_ALARM_TIMEOUT 
 #ifdef HAVE_SIGACTION
@@ -536,7 +536,7 @@ int Curl_resolv_timeout(struct connectdata *conn,
   void (*keep_sigact)(int);       /* store the old handler here */
 #endif /* HAVE_SIGNAL */
 #endif /* HAVE_SIGACTION */
-
+  volatile long timeout;
   unsigned int prev_alarm=0;
   struct SessionHandle *data = conn->data;
 #endif /* USE_ALARM_TIMEOUT */
@@ -548,6 +548,8 @@ int Curl_resolv_timeout(struct connectdata *conn,
   if (data->set.no_signal)
     /* Ignore the timeout when signals are disabled */
     timeout = 0;
+  else
+    timeout = timeoutms;
 
   if(timeout && timeout < 1000)
     /* The alarm() function only provides integer second resolution, so if
diff --git a/lib/hostip.h b/lib/hostip.h
index 08545d94e..593d2c5dc 100644
--- a/lib/hostip.h
+++ b/lib/hostip.h
@@ -166,7 +166,7 @@ int Curl_resolv(struct connectdata *conn, const char *hostname,
                 int port, struct Curl_dns_entry **dnsentry);
 int Curl_resolv_timeout(struct connectdata *conn, const char *hostname,
                         int port, struct Curl_dns_entry **dnsentry,
-                        volatile long timeout);
+                        long timeoutms);
 
 /*
  * Curl_ipvalid() checks what CURL_IPRESOLVE_* requirements that might've