Lots of work and analysis by "xbx___" in bug #1431750
authorDaniel Stenberg <daniel@haxx.se>
Thu, 23 Feb 2006 12:20:48 +0000 (12:20 +0000)
committerDaniel Stenberg <daniel@haxx.se>
Thu, 23 Feb 2006 12:20:48 +0000 (12:20 +0000)
commit6fdbb011948cc9fd2cadff04b230427cf02dbd7d
tree2aa747888250eab64d0bc6f0262790f8bfe77c26
parentd29147565c000c01a5ac20b12993c8fd726b1fa2
Lots of work and analysis by "xbx___" in bug #1431750
(http://curl.haxx.se/bug/view.cgi?id=1431750) helped me identify and fix two
different but related bugs:

1) Removing an easy handle from a multi handle before the transfer is done
   could leave a connection in the connection cache for that handle that is
   in a state that isn't suitable for re-use. A subsequent re-use could then
   read from a NULL pointer and segfault.

2) When an easy handle was removed from the multi handle, there could be an
   outstanding c-ares DNS name resolve request. When the response arrived,
   it caused havoc since the connection struct it "belonged" to could've
   been freed already.

Now Curl_done() is called when an easy handle is removed from a multi handle
pre-maturely (that is, before the transfer was complteted). Curl_done() also
makes sure to cancel all (if any) outstanding c-ares requests.
CHANGES
RELEASE-NOTES
lib/hostip.h
lib/multi.c
lib/url.c