Thomas Lopatic fixed the alarm()-based DNS timeout

diff --git a/CHANGES b/CHANGES
index 02d7b27537ce93322021eb121fd0dff5d6fd7add..8d812727f8fceed59479da0aa23cadbae17f9af0 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -6,6 +6,19 @@
 
                                   Changelog
 
+Daniel Stenberg (22 Mar 2010)
+- Thomas Lopatic fixed the alarm()-based DNS timeout:
+
+  Looking at the code of Curl_resolv_timeout() in hostip.c, I think that in
+  case of a timeout, the signal handler for SIGALRM never gets removed. I
+  think that in my case it gets executed at some point later on when execution
+  has long left Curl_resolv_timeout() or even the cURL library.
+  
+  The code that is jumped to with siglongjmp() simply sets the error message
+  to "name lookup timed out" and then returns with CURLRESOLV_ERROR. I guess
+  that instead of simply returning without cleaning up, the code should have a
+  goto that jumps to the spot right after the call to Curl_resolv().
+
 Kamil Dudka (22 Mar 2010)
 - Douglas Steinwand contributed a patch fixing insufficient initialization in
   Curl_clone_ssl_config()

diff --git a/RELEASE-NOTES b/RELEASE-NOTES
index 29ad85bde59cde60a7bed1f0b96ccef5b3d0bb20..072ad7e6b8d36aba0901cd90ff9689aaeaf8f400 100644 (file)
--- a/RELEASE-NOTES
+++ b/RELEASE-NOTES
@@ -35,6 +35,7 @@ This release includes the following bugfixes:
  o TFTP improved error codes
  o TFTP fixed TSIZE handling for uploads
  o SSL possible double free when reusing curl handle
+ o alarm()-based DNS timeout bug
 
 This release includes the following known bugs:
 
@@ -46,6 +47,6 @@ advice from friends like these:
  Steven M. Schweda, Yang Tse, Jack Zhang, Tom Donovan, Martin Hager,
  Daniel Fandrich, Patrick Monnerat, Pat Ray, Wesley Miaw, Ben Greear,
  Ryan Chan, Markus Duft, Andrei Benea, Jacob Moshenko, Daniel Johnson,
- Constantine Sapuntzakis
+ Constantine Sapuntzakis, Douglas Steinwand, Thomas Lopatic
 
         Thanks! (and sorry if I forgot to mention someone)