curl-library
Re: [PATCH v4] Add connection delay to Happy Eyeballs.
From: Björn Stenberg <bjorn_at_haxx.se>
Date: Thu, 14 Nov 2013 16:34:35 +0100
Received on 2013-11-14
Date: Thu, 14 Nov 2013 16:34:35 +0100
Steve Holme wrote:
> Looking at the code the only problem I can see is in singleipconnect() when
> the default clause gets called in "switch (error)".
This indeed looks wrong, though I fail to provoke the socket leak on any of my machines.
My attempts at getting access to a sparc machine are still unsuccessful, so I'm submitting this patch untested on sparc. It doesn't feel terribly good doing debugging via the autobuilders, but on the other hand it doesn't feel good leaving the issue unattended either.
-- Björn
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette: http://curl.haxx.se/mail/etiquette.html
- text/x-diff attachment: 0001-connect-Return-the-socket-descriptor-even-on-fail.patch
- application/pgp-signature attachment: Digital signature