Bugs item #1551412, was opened at 2006-09-03 07:19
Message generated for change (Comment added) made by zoyd2k
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=100976&aid=1551412&group_id=976

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: libcurl
Group: crash
Status: Closed
Resolution: Accepted
Priority: 5
Submitted By: Dmitry Rechkin (dortik)
Assigned to: Daniel Stenberg (bagder)
Summary: CURL timeout bug with SOCKS & segmentation fault

Initial Comment:
Hi,
I've fixed the following bugs:

1. (Known as 34) The SOCKS connection codes don't
properly acknowledge (connect) timeouts. It worked like
with timeout = 0.

2. Segmentation fault in the case when SOCKS requires
authentication and you did not specify username:password.

I've tested it on 5 servers within 20 days and seems
that everything is ok.

Please review.

----------------------------------------------------------------------

Comment By: Zoyd Wheeler (zoyd2k)
Date: 2006-10-05 19:25

Message:
Logged In: YES
user_id=1578091

I think I may be encountering the SOCKS timeout bug in my
application but there is not sufficient information here for
me to:

a) convince myself that this bug is/isn't causing my problem

b) determine the consequences of using the patch with a
blocking wait

Is there somewhere else where I can find this bug discussed
in more detail?

----------------------------------------------------------------------

Comment By: Dmitry Rechkin (dortik)
Date: 2006-09-04 01:43

Message:
Logged In: YES
user_id=1589629

My real name is Dmitry Rechkin

----------------------------------------------------------------------

Comment By: Daniel Stenberg (bagder)
Date: 2006-09-03 18:54

Message:
Logged In: YES
user_id=1110

Thanks, I've committed it just because it improves the
situation. I don't think it is a nice way of solving the
problem - as it makes a blocking wait.

Also, please beware next time that we are C89 compliant (i,e
no // comments) and please pay attention to curl style
source indenting.

If you tell me your real name, I'll give you proper credit
for the fix.

----------------------------------------------------------------------

Comment By: Dmitry Rechkin (dortik)
Date: 2006-09-03 07:20

Message:
Logged In: YES
user_id=1589629

Patch is for lib/url.c not for lib/curl.c

----------------------------------------------------------------------

You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=100976&aid=1551412&group_id=976
Received on 2006-10-06